Skip to content

[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
wants to merge 1 commit into
base: ehigham/optimise-implicit-context
Choose a base branch
from

Conversation

ehigham
Copy link
Member

@ehigham ehigham commented Oct 29, 2024

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.

@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 2d0eb42 to 5f5a835 Compare October 29, 2024 21:47
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 39595fc to 26abfd7 Compare October 29, 2024 21:48
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 5f5a835 to b47a78b Compare October 30, 2024 22:27
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 26abfd7 to 54a8581 Compare October 30, 2024 22:27
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from b47a78b to 16d94c2 Compare October 30, 2024 22:40
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 54a8581 to 7b77620 Compare October 30, 2024 22:40
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 16d94c2 to 53c2e68 Compare October 30, 2024 22:43
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 7b77620 to 575a63f Compare October 30, 2024 22:43
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 53c2e68 to 4b342b6 Compare October 30, 2024 23:40
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 575a63f to d3513a2 Compare October 30, 2024 23:40
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from ff8c38b to c161c95 Compare November 4, 2024 22:15
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from d3513a2 to d5575e9 Compare November 4, 2024 22:15
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from c161c95 to e4f8202 Compare November 19, 2024 17:09
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from d5575e9 to 2543350 Compare November 19, 2024 17:09
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from e4f8202 to 59dc4db Compare November 20, 2024 17:01
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 2543350 to 7965c8f Compare November 20, 2024 17:01
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from adf9ba2 to ae9e097 Compare February 27, 2025 22:17
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from 544766f to c1b3600 Compare March 3, 2025 17:55
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from ae9e097 to 0808f4f Compare March 3, 2025 17:55
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 0808f4f to daad5aa Compare March 17, 2025 21:54
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from c1b3600 to 9a4bec0 Compare March 17, 2025 21:54
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from daad5aa to 8bc09e3 Compare March 17, 2025 22:48
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from 1eebc10 to 7f1b00d Compare March 18, 2025 03:43
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 8bc09e3 to 08a2f5e Compare March 18, 2025 03:43
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 08a2f5e to daba4c5 Compare March 19, 2025 19:13
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from 39a329e to 56a8a6e Compare March 25, 2025 13:09
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch 2 times, most recently from 8a46a3d to c2255f3 Compare March 26, 2025 17:02
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from 83881a4 to e2a3901 Compare March 28, 2025 21:19
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch 2 times, most recently from c19e3f7 to b70969d Compare March 31, 2025 17:32
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch 2 times, most recently from 631c230 to ef519f5 Compare April 3, 2025 15:13
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from b70969d to 734adf3 Compare April 3, 2025 15:13
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from ef519f5 to e3833a3 Compare April 3, 2025 19:20
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch 2 times, most recently from 651b457 to 1c85e69 Compare April 3, 2025 19:22
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from e3833a3 to 12df0f7 Compare April 3, 2025 19:22
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 1c85e69 to b52a855 Compare April 10, 2025 15:29
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 12df0f7 to bba8006 Compare April 10, 2025 15:29
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