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 b9424b9 to 454ea3b Compare January 24, 2025 20:55
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a9fac2d to 58ea4c3 Compare January 24, 2025 20:55
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 454ea3b to 0b3c0aa Compare January 27, 2025 23:15
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 58ea4c3 to b1ccb07 Compare January 27, 2025 23:16
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 0b3c0aa to 85840f9 Compare January 29, 2025 20:24
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from b1ccb07 to ac6776b Compare January 29, 2025 20:25
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 85840f9 to d25bb41 Compare January 29, 2025 20:40
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from ac6776b to a84bcdb Compare January 29, 2025 20:40
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from d25bb41 to 4da26d3 Compare January 29, 2025 21:17
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a84bcdb to a336997 Compare January 29, 2025 21:17
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 4da26d3 to c49d75b Compare January 29, 2025 21:41
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a336997 to 01371c6 Compare January 29, 2025 21:41
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from c49d75b to 7917907 Compare January 29, 2025 22:24
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 01371c6 to 7c1cd47 Compare January 29, 2025 22:24
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 7917907 to 5a64ac4 Compare January 29, 2025 22:30
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 7c1cd47 to 1700dd4 Compare January 29, 2025 22:30
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 5a64ac4 to 1968227 Compare January 29, 2025 22:41
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 1700dd4 to 3222e11 Compare January 29, 2025 22:41
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 1968227 to fd9bde2 Compare January 29, 2025 22:54
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 3222e11 to a33d1b4 Compare January 29, 2025 22:54
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from fd9bde2 to 9f5ce3f Compare January 30, 2025 19:52
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a33d1b4 to 5ba6339 Compare January 30, 2025 19:53
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 9f5ce3f to 6195b9d Compare January 31, 2025 19:12
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 5ba6339 to 59b81a3 Compare January 31, 2025 19:13
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 6195b9d to fac7506 Compare January 31, 2025 19:27
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 59b81a3 to e2265e1 Compare January 31, 2025 19:28
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from fac7506 to 23232cd Compare January 31, 2025 21:22
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from e2265e1 to 3fdc03b Compare January 31, 2025 21:22
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