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

feat: speedup KSP extraction on Windows #112

Merged

Conversation

joel-wong-aws
Copy link
Contributor

What was the problem/requirement? (What/Why)

It was noted that progress reporting dramatically affected the speed of zip extraction using Expand-Archive in Windows PowerShell.

What was the solution? (How)

While progress reporting is nice to have, so is fast submission speed. I tested extraction times with the sample Keyframe Animation (robot head) scene from KeyShot and discovered that extraction time was sped up by 2.5x when progress reporting was not used. From previous experience, the speedup can up to 4x depending on the size of files, the machine used, etc.

We believe that customers value fast unzipping much more than progress reporting while unzipping, so I have applied the speedup by default.

What is the impact of this change?

Speedup of submission using Deadline Cloud for KeyShot.

How was this change tested?

Used this with the KeyFrame Animation scene form KeyShot and confirmed it worked.

Was this change documented?

No, not required

Is this a breaking change?

No


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@joel-wong-aws joel-wong-aws requested a review from a team as a code owner August 29, 2024 18:49
Copy link

@joel-wong-aws joel-wong-aws merged commit e9e9e34 into aws-deadline:mainline Aug 29, 2024
12 checks passed
@joel-wong-aws joel-wong-aws deleted the faster_zip_extraction branch August 29, 2024 19: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.

3 participants