Add UnionArray compute functions#8884
Conversation
Polar Signals Profiling ResultsLatest Run
Previous Runs (7)
Powered by Polar Signals Cloud |
Benchmarks: Vortex queries 📖Verdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.045x ➖, 0↑ 0↓)
datafusion / parquet (1.006x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (0.996x ➖, 0↑ 0↓)
duckdb / parquet (1.010x ➖, 0↑ 0↓)
No file size changes detected. |
6fe53ba to
a6b7e6c
Compare
96923e7 to
3233233
Compare
a6b7e6c to
1e867dd
Compare
Merging this PR will improve performance by 28.39%
Performance Changes
Tip Curious why this is faster? Comment Comparing Footnotes
|
1e867dd to
98f8f06
Compare
3233233 to
cd92358
Compare
cd92358 to
9c9debf
Compare
e32db9e to
f2427ce
Compare
dd86118 to
772b738
Compare
f2427ce to
ab33ec0
Compare
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
772b738 to
f51f2dd
Compare
ab33ec0 to
ec92eef
Compare
Rationale for this change
Depends on #8834. Tracking issue: #8769.
This is the deferred compute layer for UnionArray and remains a draft for later review.
What changes are included in this PR?
Nullable-index take remains explicitly unsupported and needs resolution before this layer is ready.
What APIs are changed? Are there any user-facing changes?
No additional public array representation APIs beyond #8834. This layer wires UnionArray into existing compute paths.