Skip to content

Support ParquetVariant through JNI#8129

Draft
AdamGS wants to merge 1 commit into
adamg/parquet-arrow-import-exportfrom
adamg/parquet-variant-jni
Draft

Support ParquetVariant through JNI#8129
AdamGS wants to merge 1 commit into
adamg/parquet-arrow-import-exportfrom
adamg/parquet-variant-jni

Conversation

@AdamGS
Copy link
Copy Markdown
Contributor

@AdamGS AdamGS commented May 27, 2026

Summary

This PR builds on top of #8125 to add support for ParquetVariant arrays, as a step towards variant support for Iceberg.

Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 27, 2026

Merging this PR will not alter performance

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 1259 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation chunked_varbinview_opt_canonical_into[(1000, 10)] 224.8 µs 188.1 µs +19.52%
Simulation chunked_varbinview_opt_into_canonical[(1000, 10)] 203.8 µs 240.4 µs -15.22%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing adamg/parquet-variant-jni (a3f3a76) with adamg/parquet-arrow-import-export (27fbac5)

Open in CodSpeed

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