Skip to content

Comments

Materializing Series with Symbolized Labels#124

Draft
francoposa wants to merge 6 commits intomainfrom
francoposa/symbolize-labels-scratch
Draft

Materializing Series with Symbolized Labels#124
francoposa wants to merge 6 commits intomainfrom
francoposa/symbolize-labels-scratch

Conversation

@francoposa
Copy link
Collaborator

No description provided.

@francoposa francoposa force-pushed the francoposa/symbolize-labels-scratch branch 2 times, most recently from 81a5bb8 to 086e786 Compare November 5, 2025 00:53
@francoposa francoposa changed the title [WIP] scratch out materializing series with symbolized labels Materializing Series with Symbolized Labels Nov 5, 2025
@francoposa
Copy link
Collaborator Author

Unclear what's happening with the PromQL acceptance test, it's untouched by this and also failing on main

@francoposa
Copy link
Collaborator Author

Current benchmarks:
benchout-materialize-symbolized-labels-086e786.txt

benchstat -col "/labels" -filter ".unit:(B-alloc-diff OR B-inuse-diff OR B/op)" benchout-materialize-symbolized-labels-086e786.txt

goos: linux
goarch: amd64
pkg: github.com/prometheus-community/parquet-common/search
cpu: AMD Ryzen 9 PRO 6950H with Radeon Graphics     
                              │    strings    │                symbols                │
                              │ B-alloc-diff  │ B-alloc-diff   vs base                │
Materialize/AllRows-16          28.09Ki ±  1%   24.73Ki ±  2%  -11.98% (p=0.000 n=10)
Materialize/AllRowsConcur4-16   28.06Ki ±  1%   24.75Ki ±  1%  -11.80% (p=0.000 n=10)
Materialize/Interleaved-16      16.06Ki ±  2%   14.43Ki ±  2%  -10.11% (p=0.000 n=10)
Materialize/Sparse-16           13.75Ki ±  1%   12.20Ki ±  2%  -11.24% (p=0.000 n=10)
Materialize/SingleRow-16        1.700Ki ± 16%   1.794Ki ± 17%        ~ (p=0.393 n=10)
Materialize/SkipChunks-16       27.99Ki ±  1%   24.91Ki ±  3%  -11.01% (p=0.000 n=10)
geomean                         14.22Ki         13.00Ki         -8.63%

                              │    strings    │                symbols                │
                              │ B-inuse-diff  │ B-inuse-diff   vs base                │
Materialize/AllRows-16          27.28Ki ± 12%   26.04Ki ±  6%        ~ (p=0.075 n=10)
Materialize/AllRowsConcur4-16   28.76Ki ±  9%   26.16Ki ±  9%   -9.04% (p=0.008 n=10)
Materialize/Interleaved-16      17.40Ki ± 20%   16.44Ki ±  9%        ~ (p=0.494 n=10)
Materialize/Sparse-16           17.20Ki ± 12%   14.44Ki ± 12%  -16.05% (p=0.008 n=10)
Materialize/SingleRow-16        1.800Ki ± 51%   2.880Ki ± 39%        ~ (p=0.085 n=10)
Materialize/SkipChunks-16       29.28Ki ±  8%   26.04Ki ± 16%  -11.06% (p=0.011 n=10)
geomean                         15.21Ki         15.16Ki         -0.33%

                              │   strings    │               symbols                │
                              │     B/op     │     B/op      vs base                │
Materialize/AllRows-16          2.753Mi ± 0%   2.409Mi ± 0%  -12.50% (p=0.000 n=10)
Materialize/AllRowsConcur4-16   2.756Mi ± 0%   2.409Mi ± 0%  -12.56% (p=0.000 n=10)
Materialize/Interleaved-16      1.584Mi ± 0%   1.415Mi ± 0%  -10.68% (p=0.000 n=10)
Materialize/Sparse-16           1.340Mi ± 0%   1.208Mi ± 0%   -9.84% (p=0.000 n=10)
Materialize/SingleRow-16        175.9Ki ± 2%   176.6Ki ± 1%        ~ (p=0.143 n=10)
Materialize/SkipChunks-16       2.755Mi ± 0%   2.412Mi ± 0%  -12.46% (p=0.000 n=10)
geomean                         1.403Mi        1.266Mi        -9.72%

Signed-off-by: francoposa <franco@francoposa.io>
Signed-off-by: francoposa <franco@francoposa.io>
Signed-off-by: francoposa <franco@francoposa.io>
Signed-off-by: francoposa <franco@francoposa.io>
Signed-off-by: francoposa <franco@francoposa.io>
@francoposa francoposa force-pushed the francoposa/symbolize-labels-scratch branch from 66b7bed to de3dfd4 Compare November 5, 2025 02:17
Signed-off-by: francoposa <franco@francoposa.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant