Add UnionArray compute functions#8884
Conversation
Polar Signals Profiling ResultsLatest Run
Previous Runs (6)
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.040x ➖, 0↑ 0↓)
datafusion / parquet (0.962x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (1.017x ➖, 0↑ 0↓)
duckdb / parquet (1.007x ➖, 0↑ 0↓)
No file size changes detected. |
6fe53ba to
a6b7e6c
Compare
96923e7 to
3233233
Compare
a6b7e6c to
1e867dd
Compare
Merging this PR will degrade performance by 14.37%
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
1e867dd to
98f8f06
Compare
3233233 to
cd92358
Compare
cd92358 to
9c9debf
Compare
e32db9e to
f2427ce
Compare
dd86118 to
772b738
Compare
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
f2427ce to
ab33ec0
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.