-
Notifications
You must be signed in to change notification settings - Fork 250
[query] make local and remote tmp settable on backend #14748
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
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
Conversation
This file contains hidden or 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
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
39a329e
to
56a8a6e
Compare
8a46a3d
to
c2255f3
Compare
83881a4
to
e2a3901
Compare
c19e3f7
to
b70969d
Compare
631c230
to
ef519f5
Compare
b70969d
to
734adf3
Compare
ef519f5
to
e3833a3
Compare
651b457
to
1c85e69
Compare
e3833a3
to
12df0f7
Compare
1c85e69
to
b52a855
Compare
12df0f7
to
bba8006
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.