Skip to content

Conversation

@ehigham
Copy link
Member

@ehigham ehigham commented Oct 29, 2024

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.

This change has no security impact

@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 0fab4fd to 9d9d912 Compare August 1, 2025 22:16
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from f27cd7a to 705f949 Compare August 1, 2025 22:16
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 9d9d912 to 2e55e71 Compare August 2, 2025 04:21
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 705f949 to 1abe76b Compare August 2, 2025 04:21
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from 2e55e71 to c97beef Compare August 2, 2025 04:59
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 1abe76b to 0058c0d Compare August 2, 2025 04:59
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from c97beef to a35e964 Compare August 2, 2025 05:02
@ehigham ehigham force-pushed the ehigham/optimise-implicit-context branch from 0058c0d to 9d363a1 Compare August 2, 2025 05:02
Base automatically changed from ehigham/optimise-implicit-context to main August 2, 2025 06:45
@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a35e964 to 1b2e3ed Compare August 4, 2025 16:57
Copy link
Collaborator

@chrisvittal chrisvittal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this.

@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch 3 times, most recently from 11d12b1 to a492cc5 Compare August 5, 2025 19:08
Copy link
Member

@patrick-schultz patrick-schultz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this too

@ehigham ehigham force-pushed the ehigham/tmpdir-per-query branch from a492cc5 to 4c46605 Compare September 15, 2025 20:42
@hail-ci-robot hail-ci-robot merged commit 3a6917a into main Sep 15, 2025
2 checks passed
@hail-ci-robot hail-ci-robot deleted the ehigham/tmpdir-per-query branch September 15, 2025 21:36
kush-chandra pushed a commit to kush-chandra/hail that referenced this pull request Sep 17, 2025
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.

This change has no security impact
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.

5 participants