Use cached prism binary without re-unzipping by default. #34616
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously, when we have cache enabled for prism binary/zip and both binary and zip file in cache,
PrismJobServer
will still unzip the prism zip file. This leads to the error ofText file busy
when we try to run a prism runner the second time in colab (#33623 (comment)).This PR fixes this problem and adds an array of unit tests.
addresses #33623
Notice that there is another problem in the original issue regarding running the prism runner multiple times results in multiple prism processes in colab. There will be a follow-up PR (#34623) for that.