Skip to content

Conversation

@haohuaijin
Copy link
Contributor

Does this PR closes an open issue or discussion?

  • Closes #.

What changes are included in this PR?

re export the BtrBlocksCompressorBuilder in the vortex crate

What is the rationale for this change?

i see we export BtrBlocksCompressor in vortex crate, seem like no reason not export it Builder struct

How is this change tested?

Are there any user-facing changes?

@joseph-isaacs joseph-isaacs enabled auto-merge (squash) February 11, 2026 16:18
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 11, 2026

Merging this PR will improve performance by 35.59%

⚡ 5 improved benchmarks
✅ 1130 untouched benchmarks
⏩ 1268 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation take_map[(0.05, 1.0)] 4.1 ms 3.1 ms +31.32%
Simulation take_map[(0.05, 0.1)] 792 µs 592.1 µs +33.77%
Simulation take_map[(0.1, 0.5)] 2.8 ms 2 ms +35.59%
Simulation take_map[(0.1, 1.0)] 4.7 ms 3.5 ms +33.92%
Simulation take_map[(0.1, 0.1)] 1,118.8 µs 924.8 µs +20.99%

Comparing haohuaijin:re-export-builder (103a1e6) with develop (aa55054)

Open in CodSpeed

Footnotes

  1. 1268 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.

@AdamGS AdamGS added the changelog/fix A bug fix label Feb 11, 2026
@joseph-isaacs joseph-isaacs merged commit dbe0ff5 into vortex-data:develop Feb 11, 2026
47 of 49 checks passed
@haohuaijin haohuaijin deleted the re-export-builder branch February 11, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants