Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(interactive): Fixed bug that wasted disk space #3929

Merged
merged 3 commits into from
Jun 18, 2024

Conversation

yqylh
Copy link
Contributor

@yqylh yqylh commented Jun 17, 2024

What do these changes do?

The interactive service generates n cache files named neo4j-xxx in the /tmp/ directory during execution, each with a size of at least 0.5GB. These files do not get deleted when the process ends. Here, n represents the number of times the interactive server has been started or restarted.

This issue was due to the previously used termination method for the compiler process, which utilized the terminate() function. Since terminate() does not facilitate a normal exit, it led to the compiler's inability to clean up the cache files it created.

Transitioning to sending the SIGINT signal ensures that the compiler exits gracefully and performs the necessary cache cleanup. And sleep for up to 10 seconds to wait for the compiler process to stop

## What do these changes do?
The interactive service generates n cache files named neo4j-xxx in the /tmp/ directory during execution, each with a size of at least **0.5GB**. These files do not get deleted when the process ends. Here, n represents the number of times the interactive server has been started or restarted.

This issue was due to the previously used termination method for the compiler process, which utilized the **terminate()** function. Since terminate() does not facilitate a normal exit, it led to the compiler's inability to clean up the cache files it created.

Transitioning to sending the **SIGINT signal** ensures that the compiler exits gracefully and performs the necessary cache cleanup.

Sleep for up to 10 seconds to wait for the compiler process to stop
Copy link
Collaborator

@zhanglei1949 zhanglei1949 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@liulx20 liulx20 self-requested a review June 17, 2024 09:03
@zhanglei1949 zhanglei1949 merged commit f1cf69d into alibaba:main Jun 18, 2024
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants