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

Reuse a temp dir per device #40

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Reuse a temp dir per device #40

merged 1 commit into from
Jan 9, 2024

Conversation

Dr-Emann
Copy link
Owner

@Dr-Emann Dr-Emann commented Jan 9, 2024

Try to use the system tempdir first, then make a tempdir per device the root directories reside on.

This has several benefits:

  • Most of the time, we'll use the common system tempdir entirely
  • We will now exclude the tempdir we use for each root: it used to be possible for the temp files to be picked up by the reader thread: this is now only possible if we cross into a different device than all root dirs.

Try to use the system tempdir first, then make a tempdir per device the
root directories reside on.

This has several benefits:
- Most of the time, we'll use the common system tempdir entirely
- We will now exclude the tempdir we use for each root: it used to be
  possible for the temp files to be picked up by the reader thread: this
  is now only possible if we cross into a different device than all root
  dirs.
@Dr-Emann Dr-Emann merged commit c413d8e into main Jan 9, 2024
2 checks passed
@Dr-Emann Dr-Emann deleted the common_tempdirs branch January 9, 2024 00:56
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.

1 participant