Skip to content

feat[vortex-array]: support converting arrow REE dtype to vortex#6784

Merged
joseph-isaacs merged 1 commit intodevelopfrom
asubiotto/reedtype
Mar 5, 2026
Merged

feat[vortex-array]: support converting arrow REE dtype to vortex#6784
joseph-isaacs merged 1 commit intodevelopfrom
asubiotto/reedtype

Conversation

@asubiotto
Copy link
Contributor

Summary

Adds a match arm to convert arrow REE dtypes by recursing into the value type. This was previously unsupported.

Testing

@asubiotto asubiotto added the changelog/feature A new feature label Mar 4, 2026
@asubiotto asubiotto requested a review from gatesn March 4, 2026 17:06
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 4, 2026

Merging this PR will improve performance by 11.36%

⚡ 1 improved benchmark
✅ 979 untouched benchmarks
⏩ 1466 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation map_each[BufferMut<i32>, 128] 858.1 ns 770.6 ns +11.36%

Comparing asubiotto/reedtype (bbca2bd) with develop (df23c37)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Signed-off-by: Alfonso Subiotto Marques <alfonso.subiotto@polarsignals.com>
@asubiotto asubiotto force-pushed the asubiotto/reedtype branch from 2663b98 to bbca2bd Compare March 4, 2026 20:44
@joseph-isaacs joseph-isaacs merged commit c361a3a into develop Mar 5, 2026
56 checks passed
@joseph-isaacs joseph-isaacs deleted the asubiotto/reedtype branch March 5, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants