Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Conversation

@AmelBawa-msft
Copy link
Contributor

@AmelBawa-msft AmelBawa-msft commented Sep 18, 2024

Summary of the pull request

Code changes in this PR allows copying packages from out-of-proc to in-proc in parallel offering x1.44 speedup on average per search query. Those values are based on 28 data points executed before and after the change.

⚠️ Those numbers may be different on different machines and depends on hardware and software variables.

  Before (ms) After (ms) Speedup
SUM 46,613 32,272 1.444379
AVG 1,726 1,195 1.444379

References and relevant issues

Detailed description of the pull request / Additional comments

Validation steps performed

PR checklist

@AmelBawa-msft AmelBawa-msft requested a review from a team September 18, 2024 22:17
@AmelBawa-msft AmelBawa-msft added the Needs-Second Pull request that needs another approval label Sep 23, 2024
@krschau krschau removed the Needs-Second Pull request that needs another approval label Sep 23, 2024
@AmelBawa-msft AmelBawa-msft merged commit 19b988c into main Sep 24, 2024
@AmelBawa-msft AmelBawa-msft deleted the user/amelbawa/3655 branch September 24, 2024 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Searching for apps takes a long time

4 participants