Skip to content

FEAT: Add CI perf baseline regression detection & insertmanyvalues benchmark #527

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.

Sign up for GitHub

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

Open
bewithgaurav wants to merge 7 commits into
base: main
Choose a base branch
from
Open

FEAT: Add CI perf baseline regression detection & insertmanyvalues benchmark #527

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f419d4c
Revamp perf benchmarks: insertmanyvalues scenario, baseline compariso…
bewithgaurav Apr 17, 2026
422767b
Add benchmark runs to macOS CI pipeline
bewithgaurav Apr 20, 2026
c5098e1
Add PERF: as allowed PR title prefix
bewithgaurav Apr 20, 2026
a77fd5d
Fix macOS benchmark: install ODBC driver for pyodbc, use tcp:127.0.0.…
bewithgaurav Apr 20, 2026
a3eb8e3
Fix benchmark baseline: download before run, rename to expected path,…
bewithgaurav Apr 20, 2026
1670065
Fix review comments: resource cleanup, baseline subfolder search, Lin…
bewithgaurav Apr 20, 2026
ebf8934
Fix macOS benchmark: add Microsoft brew tap before installing msodbcs…
bewithgaurav Apr 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.MD
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,9 @@ STYLE: (short-description)
> For Refactor, without any feature changes
REFACTOR: (short-description)

> For performance improvements
PERF: (short-description)

> For release related changes, without any feature changes
RELEASE: #<RELEASE_VERSION> (short-description)

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pr-format-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
// Validate title prefix for all contributors
const validTitlePrefixes = [
'FEAT:', 'CHORE:', 'FIX:', 'DOC:', 'STYLE:', 'REFACTOR:',
'RELEASE:'
'PERF:', 'RELEASE:'
];

const hasValidPrefix = validTitlePrefixes.some(prefix =>
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ If you are a Microsoft organization member (internal contributor):

All pull requests must include:

- **Valid Title Prefix**: Your PR title must start with one of: `FEAT:`, `CHORE:`, `FIX:`, `DOC:`, `STYLE:`, `REFACTOR:`, or `RELEASE:`
- **Valid Title Prefix**: Your PR title must start with one of: `FEAT:`, `CHORE:`, `FIX:`, `DOC:`, `STYLE:`, `REFACTOR:`, `PERF:`, or `RELEASE:`
- **Meaningful Summary**: Include a clear description of your changes under the "### Summary" section in the PR description (minimum 10 characters)
- **Issue/Work Item Link** (only one required):
- External contributors: Link to a GitHub issue
Expand Down
Loading
Loading