-
Notifications
You must be signed in to change notification settings - Fork 248
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
[query] make local and remote tmp settable on backend #14748
Open
ehigham
wants to merge
1
commit into
ehigham/optimise-implicit-context
Choose a base branch
from
ehigham/tmpdir-per-query
base: ehigham/optimise-implicit-context
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[query] make local and remote tmp settable on backend #14748
ehigham
wants to merge
1
commit into
ehigham/optimise-implicit-context
from
ehigham/tmpdir-per-query
+81
−21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Oct 29, 2024
This was referenced Oct 29, 2024
2d0eb42
to
5f5a835
Compare
39595fc
to
26abfd7
Compare
5f5a835
to
b47a78b
Compare
26abfd7
to
54a8581
Compare
b47a78b
to
16d94c2
Compare
54a8581
to
7b77620
Compare
16d94c2
to
53c2e68
Compare
7b77620
to
575a63f
Compare
53c2e68
to
4b342b6
Compare
575a63f
to
d3513a2
Compare
ff8c38b
to
c161c95
Compare
d3513a2
to
d5575e9
Compare
c161c95
to
e4f8202
Compare
d5575e9
to
2543350
Compare
e4f8202
to
59dc4db
Compare
2543350
to
7965c8f
Compare
This was referenced Dec 5, 2024
23232cd
to
6f3042d
Compare
3fdc03b
to
dd84033
Compare
6f3042d
to
62c51bc
Compare
dd84033
to
3db6757
Compare
62c51bc
to
d658770
Compare
3db6757
to
bd63550
Compare
d658770
to
8f664ef
Compare
bd63550
to
1fb53a2
Compare
8f664ef
to
b3cce7a
Compare
1fb53a2
to
fc7b02f
Compare
b3cce7a
to
0544342
Compare
fc7b02f
to
12c6632
Compare
12c6632
to
adcedfa
Compare
0544342
to
573437a
Compare
adcedfa
to
adf9ba2
Compare
573437a
to
6b2b978
Compare
adf9ba2
to
ae9e097
Compare
544766f
to
c1b3600
Compare
ae9e097
to
0808f4f
Compare
0808f4f
to
daad5aa
Compare
c1b3600
to
9a4bec0
Compare
daad5aa
to
8bc09e3
Compare
1eebc10
to
7f1b00d
Compare
8bc09e3
to
08a2f5e
Compare
08a2f5e
to
daba4c5
Compare
7f1b00d
to
39a329e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change Description
Refactors temporary directory handling in Hail's backend system by introducing abstract properties for local and remote temporary directories. This moves the temporary directory management from the HailContext into the backend implementations, providing a more consistent interface for handling temporary storage locations across different backend types.
Security Assessment
This change has no security impact
Impact Description
Low-level refactoring of temporary directory management code. No changes to how temporary directories are created or accessed, only reorganizing where this functionality lives in the codebase.