Skip to content

[Internal] Query: Fixes Perf Tests#5149

Merged
microsoft-github-policy-service[bot] merged 18 commits intomasterfrom
users/mayapainter/benchmark
May 16, 2025
Merged

[Internal] Query: Fixes Perf Tests#5149
microsoft-github-policy-service[bot] merged 18 commits intomasterfrom
users/mayapainter/benchmark

Conversation

@Maya-Painter
Copy link
Copy Markdown
Contributor

@Maya-Painter Maya-Painter commented Apr 28, 2025

Description

Updates perf tests to:
- Set up benchmark collections
- Compare text and binary transport serialization
- Allow multiple queries to be tested at once
- Generate formatted CSV files
- And other misc. improvements

adityasa
adityasa previously approved these changes May 16, 2025
Copy link
Copy Markdown
Contributor

@adityasa adityasa left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Copy Markdown
Contributor

@adityasa adityasa left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Copy Markdown
Contributor

@neildsh neildsh left a comment

Choose a reason for hiding this comment

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

This test likely needs to be rewritten to use benchmark.net and moved to the performance test dll

@Maya-Painter Maya-Painter added the auto-merge Enables automation to merge PRs label May 16, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot enabled auto-merge (squash) May 16, 2025 21:45
@microsoft-github-policy-service microsoft-github-policy-service Bot merged commit d5d16df into master May 16, 2025
26 checks passed
@microsoft-github-policy-service microsoft-github-policy-service Bot deleted the users/mayapainter/benchmark branch May 16, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Enables automation to merge PRs QUERY

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants