-
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
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
October 29, 2024 21:47
2d0eb42
to
5f5a835
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
October 29, 2024 21:48
39595fc
to
26abfd7
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
October 30, 2024 22:27
5f5a835
to
b47a78b
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
October 30, 2024 22:27
26abfd7
to
54a8581
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
October 30, 2024 22:40
b47a78b
to
16d94c2
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
October 30, 2024 22:40
54a8581
to
7b77620
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
October 30, 2024 22:43
16d94c2
to
53c2e68
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
October 30, 2024 22:43
7b77620
to
575a63f
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
October 30, 2024 23:40
53c2e68
to
4b342b6
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
October 30, 2024 23:40
575a63f
to
d3513a2
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
2 times, most recently
from
November 4, 2024 22:15
ff8c38b
to
c161c95
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
November 4, 2024 22:15
d3513a2
to
d5575e9
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
November 19, 2024 17:09
c161c95
to
e4f8202
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
November 19, 2024 17:09
d5575e9
to
2543350
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
November 20, 2024 17:01
e4f8202
to
59dc4db
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
November 20, 2024 17:01
2543350
to
7965c8f
Compare
This was referenced Dec 5, 2024
1 task
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 24, 2025 20:55
b9424b9
to
454ea3b
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 24, 2025 20:55
a9fac2d
to
58ea4c3
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 27, 2025 23:15
454ea3b
to
0b3c0aa
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 27, 2025 23:16
58ea4c3
to
b1ccb07
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 20:24
0b3c0aa
to
85840f9
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 20:25
b1ccb07
to
ac6776b
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 20:40
85840f9
to
d25bb41
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 20:40
ac6776b
to
a84bcdb
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 21:17
d25bb41
to
4da26d3
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 21:17
a84bcdb
to
a336997
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 21:41
4da26d3
to
c49d75b
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 21:41
a336997
to
01371c6
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 22:24
c49d75b
to
7917907
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 22:24
01371c6
to
7c1cd47
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 22:30
7917907
to
5a64ac4
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 22:30
7c1cd47
to
1700dd4
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 22:41
5a64ac4
to
1968227
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 22:41
1700dd4
to
3222e11
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 29, 2025 22:54
1968227
to
fd9bde2
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 29, 2025 22:54
3222e11
to
a33d1b4
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 30, 2025 19:52
fd9bde2
to
9f5ce3f
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 30, 2025 19:53
a33d1b4
to
5ba6339
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 31, 2025 19:12
9f5ce3f
to
6195b9d
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 31, 2025 19:13
5ba6339
to
59b81a3
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 31, 2025 19:27
6195b9d
to
fac7506
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 31, 2025 19:28
59b81a3
to
e2265e1
Compare
ehigham
force-pushed
the
ehigham/optimise-implicit-context
branch
from
January 31, 2025 21:22
fac7506
to
23232cd
Compare
ehigham
force-pushed
the
ehigham/tmpdir-per-query
branch
from
January 31, 2025 21:22
e2265e1
to
3fdc03b
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.