perf: align buffer to Alignment::DEFAULT_ALIGNMENT = = Alignment::new(256)#8490
perf: align buffer to Alignment::DEFAULT_ALIGNMENT = = Alignment::new(256)#8490joseph-isaacs wants to merge 10 commits into
Alignment::DEFAULT_ALIGNMENT = = Alignment::new(256)#8490Conversation
Polar Signals Profiling ResultsLatest Run
Previous Runs (2)
Powered by Polar Signals Cloud |
Benchmarks: CompressionVortex (geomean): 1.005x ➖ How to read Verdict and Engines
unknown / unknown (1.088x ➖, 3↑ 63↓)
|
Merging this PR will not alter performance
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | varbinview_large |
130.8 µs | 393.6 µs | -66.76% |
| ❌ | Simulation | from_iter_bit_buffer[128] |
4.4 µs | 5.1 µs | -14.1% |
| ❌ | Simulation | chunked_varbinview_canonical_into[(100, 100)] |
223.8 µs | 259.6 µs | -13.78% |
| ⚡ | Simulation | runend_compress_u32 |
4.8 ms | 3.8 ms | +24.28% |
| ⚡ | Simulation | chunked_bool_canonical_into[(1000, 10)] |
33 µs | 26.9 µs | +22.64% |
| ⚡ | Simulation | chunked_varbinview_opt_canonical_into[(1000, 10)] |
206.7 µs | 170.8 µs | +21.02% |
| ⚡ | Simulation | chunked_varbinview_opt_into_canonical[(1000, 10)] |
220.4 µs | 183.7 µs | +20% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[2048] |
427.8 ns | 369.4 ns | +15.79% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[128] |
244.4 ns | 215.3 ns | +13.55% |
| ⚡ | Simulation | map_each[BufferMut<i32>, 128] |
495.3 ns | 436.9 ns | +13.35% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[1024] |
304.7 ns | 275.6 ns | +10.58% |
| ⚡ | Simulation | encode_varbin[(1000, 512)] |
220.3 µs | 199.5 µs | +10.43% |
| ⚡ | Simulation | slice_vortex_buffer[1024] |
929.7 ns | 842.2 ns | +10.39% |
| ⚡ | Simulation | slice_vortex_buffer[16384] |
929.7 ns | 842.2 ns | +10.39% |
| ⚡ | Simulation | slice_vortex_buffer[2048] |
929.7 ns | 842.2 ns | +10.39% |
| ⚡ | Simulation | slice_vortex_buffer[128] |
929.7 ns | 842.2 ns | +10.39% |
| ⚡ | Simulation | slice_vortex_buffer[65536] |
929.7 ns | 842.2 ns | +10.39% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ji/over-align-buffer-default (bfc91e8) with develop (02e085f)
Benchmarks: Appian on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.986x ➖, 0↑ 0↓)
datafusion / parquet (0.983x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (0.984x ➖, 0↑ 0↓)
duckdb / parquet (0.988x ➖, 0↑ 0↓)
duckdb / duckdb (0.983x ➖, 0↑ 0↓)
File Size Changes (3 files changed, -0.0% overall, 1↑ 2↓)
Totals:
|
Benchmarks: TPC-H SF=10 on S3Verdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.930x ➖, 0↑ 0↓)
datafusion / vortex-compact (1.010x ➖, 2↑ 4↓)
datafusion / parquet (0.893x ➖, 3↑ 0↓)
duckdb / vortex-file-compressed (0.956x ➖, 0↑ 0↓)
duckdb / vortex-compact (0.965x ➖, 0↑ 0↓)
duckdb / parquet (1.001x ➖, 0↑ 0↓)
|
Benchmarks: TPC-H SF=1 on S3Verdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.998x ➖, 1↑ 4↓)
datafusion / vortex-compact (0.850x ➖, 2↑ 0↓)
datafusion / parquet (0.811x ➖, 10↑ 2↓)
duckdb / vortex-file-compressed (0.958x ➖, 0↑ 0↓)
duckdb / vortex-compact (1.035x ➖, 0↑ 3↓)
duckdb / parquet (0.976x ➖, 0↑ 0↓)
|
Benchmarks: TPC-H SF=10 on NVMEVerdict: No clear signal (medium confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.207x ❌, 0↑ 22↓)
datafusion / vortex-compact (1.150x ❌, 0↑ 20↓)
datafusion / parquet (1.187x ❌, 0↑ 19↓)
datafusion / arrow (1.057x ➖, 4↑ 12↓)
duckdb / vortex-file-compressed (1.028x ➖, 0↑ 1↓)
duckdb / vortex-compact (1.005x ➖, 0↑ 0↓)
duckdb / parquet (1.035x ➖, 0↑ 0↓)
duckdb / duckdb (1.008x ➖, 0↑ 0↓)
File Size Changes (26 files changed, +0.0% overall, 12↑ 14↓)
Totals:
|
Benchmarks: TPC-DS SF=1 on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.983x ➖, 3↑ 1↓)
datafusion / vortex-compact (0.991x ➖, 1↑ 3↓)
datafusion / parquet (0.992x ➖, 1↑ 3↓)
duckdb / vortex-file-compressed (0.998x ➖, 8↑ 5↓)
duckdb / vortex-compact (0.982x ➖, 7↑ 2↓)
duckdb / parquet (1.011x ➖, 0↑ 2↓)
duckdb / duckdb (1.038x ➖, 1↑ 5↓)
File Size Changes (9 files changed, +0.0% overall, 5↑ 4↓)
Totals:
|
Benchmarks: Clickbench on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.047x ➖, 1↑ 7↓)
datafusion / parquet (1.036x ➖, 0↑ 2↓)
duckdb / vortex-file-compressed (0.982x ➖, 6↑ 2↓)
duckdb / parquet (1.007x ➖, 0↑ 0↓)
duckdb / duckdb (1.003x ➖, 0↑ 0↓)
File Size Changes (104 files changed, +0.0% overall, 54↑ 50↓)
Totals:
|
Benchmarks: PolarSignals ProfilingVortex (geomean): 0.899x ✅ How to read Verdict and Engines
datafusion / vortex-file-compressed (0.899x ✅, 5↑ 0↓)
No file size changes detected. |
Benchmarks: TPC-H SF=1 on NVMEVerdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.009x ➖, 0↑ 0↓)
datafusion / vortex-compact (0.960x ➖, 1↑ 0↓)
datafusion / parquet (0.967x ➖, 5↑ 2↓)
datafusion / arrow (0.853x ✅, 15↑ 0↓)
duckdb / vortex-file-compressed (0.946x ➖, 4↑ 0↓)
duckdb / vortex-compact (0.970x ➖, 1↑ 0↓)
duckdb / parquet (0.978x ➖, 1↑ 1↓)
duckdb / duckdb (0.980x ➖, 0↑ 0↓)
File Size Changes (9 files changed, -0.1% overall, 4↑ 5↓)
Totals:
|
Benchmarks: FineWeb S3Verdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.024x ➖, 0↑ 1↓)
datafusion / vortex-compact (0.943x ➖, 0↑ 0↓)
datafusion / parquet (0.974x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (1.060x ➖, 0↑ 1↓)
duckdb / vortex-compact (0.969x ➖, 0↑ 1↓)
duckdb / parquet (0.969x ➖, 0↑ 0↓)
|
Benchmarks: FineWeb NVMeVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.889x ✅, 6↑ 0↓)
datafusion / vortex-compact (0.888x ✅, 6↑ 0↓)
datafusion / parquet (0.884x ✅, 7↑ 0↓)
duckdb / vortex-file-compressed (0.972x ➖, 0↑ 0↓)
duckdb / vortex-compact (0.965x ➖, 1↑ 0↓)
duckdb / parquet (0.912x ➖, 2↑ 0↓)
File Size Changes (1 files changed, +0.0% overall, 1↑ 0↓)
Totals:
|
Benchmarks: Statistical and Population GeneticsVerdict: No clear signal (low confidence) How to read Verdict and Engines
duckdb / vortex-file-compressed (0.999x ➖, 0↑ 0↓)
duckdb / vortex-compact (1.003x ➖, 0↑ 0↓)
duckdb / parquet (0.991x ➖, 0↑ 0↓)
File Size Changes (1 files changed, +0.0% overall, 1↑ 0↓)
Totals:
|
794fc4c to
9810f14
Compare
e5baa81 to
6f42904
Compare
Benchmarks: Random AccessVortex (geomean): 1.097x ➖ How to read Verdict and Engines
unknown / unknown (1.102x ❌, 0↑ 12↓)
|
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Signed-off-by: Robert Kruszewski <github@robertk.io>
9208044 to
5634a7a
Compare
Default alignment of new buffer to 256