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

[query] make local and remote tmp settable on backend #14748

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/optimise-implicit-context branch from 23232cd to 6f3042d Compare February 3, 2025 17:21
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 3fdc03b to dd84033 Compare February 3, 2025 17:21
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 6f3042d to 62c51bc Compare February 3, 2025 18:01
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from dd84033 to 3db6757 Compare February 3, 2025 18:02
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 62c51bc to d658770 Compare February 3, 2025 20:51
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 3db6757 to bd63550 Compare February 3, 2025 20:52
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from d658770 to 8f664ef Compare February 3, 2025 23:07
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from bd63550 to 1fb53a2 Compare February 3, 2025 23:07
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 8f664ef to b3cce7a Compare February 4, 2025 22:44
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 1fb53a2 to fc7b02f Compare February 4, 2025 22:44
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from b3cce7a to 0544342 Compare February 6, 2025 18:44
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from fc7b02f to 12c6632 Compare February 6, 2025 18:44
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 12c6632 to adcedfa Compare February 20, 2025 16:50
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 0544342 to 573437a Compare February 20, 2025 16:50
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from adcedfa to adf9ba2 Compare February 20, 2025 17:28
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 573437a to 6b2b978 Compare February 20, 2025 17:28
@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 from 7f1b00d to 39a329e Compare March 19, 2025 19:13
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