8000 fix[ci]: use correct protoc by joseph-isaacs · Pull Request #6884 · vortex-data/vortex · GitHub
[go: up one dir, main page]

Skip to content

fix[ci]: use correct protoc#6884

Merged
joseph-isaacs merged 2 commits intodevelopfrom
ji/more-cores-ci
Mar 11, 2026
Merged

fix[ci]: use correct protoc#6884
joseph-isaacs merged 2 commits intodevelopfrom
ji/more-cores-ci

Conversation

@joseph-isaacs
Copy link
Contributor

use correct protoc

@joseph-isaacs joseph-isaacs enabled auto-merge (squash) March 11, 2026 13:49
@joseph-isaacs joseph-isaacs requested a review from myrrc March 11, 2026 13:49
@myrrc myrrc added changelog/chore A trivial change chore labels Mar 11, 2026
@codspeed-hq
Copy link
codspeed-hq bot commented Mar 11, 2026

Merging this PR will degrade performance by 20.91%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 5 regressed benchmarks
✅ 1024 untouched benchmarks
⏩ 1466 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation dict_canonicalize_zipfian[16, 1000] 56.2 µs 63.1 µs -10.96%
Simulation patched_take_200k_first_chunk_only 4.8 ms 5.4 ms -10.7%
Simulation patched_take_200k_dispersed 4.7 ms 5.6 ms -16.55%
Simulation take_200k_dispersed 3.6 ms 4.5 ms -19.59%
Simulation take_200k_first_chunk_only 3.3 ms 4.2 ms -20.91%

Comparing ji/more-cores-ci (afef41f) with develop (11a2733)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@joseph-isaacs joseph-isaacs merged commit b606e1a into develop Mar 11, 2026
65 of 68 checks passed
@joseph-isaacs joseph-isaacs deleted the ji/more-cores-ci branch March 11, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0