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

Refactor zstash #370

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Refactor zstash #370

wants to merge 1 commit into from

Conversation

forsyth2
Copy link
Collaborator

@forsyth2 forsyth2 commented Mar 7, 2025

Extremely early, experimental draft of what a zstash refactor would look like. Specifically, the refactor would store as much state as possible in an object rather than passing around many variables (especially global variables).

@forsyth2 forsyth2 self-assigned this Mar 7, 2025
@forsyth2
Copy link
Collaborator Author

forsyth2 commented Mar 7, 2025

Advantages:

  • State could be easily "snapshotted". In particular, it would be easier to track state of Globus transfers (as in Non block testing fix #363).
  • Easier handling of HPSS (e.g., the HPSSType Enum of this PR allows us to immediately see which HPSS scenario we're in rather than repeatedly parsing the HPSS path

Obstacles:

  • Unclear at the moment what Config class is for and if the CommandInfo of this PR would be better off combined with it
  • Extensive changes to parameter passing.

@forsyth2 forsyth2 mentioned this pull request Mar 7, 2025
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