Skip to content

Conversation

@AdamGS
Copy link
Contributor

@AdamGS AdamGS commented Jan 20, 2026

Instead of using a feature just for an error type, we introduce a new error type that helps us localize the functionality into vortex-python, similarly to the PyVortex type.

Like #6059, this should also help with CI and simplify our internal dependency matrix.

@AdamGS AdamGS requested review from danking and gatesn January 20, 2026 16:15
@AdamGS AdamGS added break Release label indicating a breaking API change fix labels Jan 20, 2026
@AdamGS AdamGS force-pushed the adamg/remove-python-error-feature branch from 1ac3c1c to ac3d92d Compare January 20, 2026 16:16
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@AdamGS AdamGS force-pushed the adamg/remove-python-error-feature branch from ac3d92d to 46c19a5 Compare January 20, 2026 16:17
@AdamGS AdamGS changed the title Remove the 'python' feature from vortex-error Remove the python feature from vortex-error Jan 20, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 21, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 16.5%

Comparing adamg/remove-python-error-feature (af48c30) with develop (e5649fc)

Summary

⚡ 7 improved benchmarks
❌ 2 regressed benchmarks
✅ 1245 untouched benchmarks
⏩ 1254 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation canonical_into_non_nullable[(10000, 100, 0.1)] 4.5 ms 3.7 ms +22.18%
Simulation canonical_into_non_nullable[(10000, 100, 0.0)] 2.7 ms 1.9 ms +42.36%
Simulation canonical_into_non_nullable[(10000, 100, 0.01)] 2.9 ms 2.1 ms +37.72%
Simulation canonical_into_nullable[(10000, 100, 0.0)] 4.1 ms 4.9 ms -16.5%
Simulation canonical_into_nullable[(10000, 10, 0.0)] 444.9 µs 529.3 µs -15.96%
Simulation into_canonical_non_nullable[(10000, 100, 0.1)] 4.6 ms 3.7 ms +21.6%
Simulation into_canonical_non_nullable[(10000, 100, 0.0)] 2.7 ms 1.9 ms +42.13%
Simulation into_canonical_non_nullable[(10000, 100, 0.01)] 3 ms 2.2 ms +36.99%
Simulation into_canonical_nullable[(10000, 100, 0.0)] 5.2 ms 4.4 ms +18.61%

Footnotes

  1. 1254 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.

@AdamGS AdamGS enabled auto-merge (squash) January 21, 2026 15:02
impl PyStructArray {
/// Returns the given field of the struct array.
pub fn field(self_: PyRef<'_, Self>, name: &str) -> PyResult<PyArrayRef> {
pub fn field(self_: PyRef<'_, Self>, name: &str) -> Result<PyArrayRef, PyVortexError> {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you type alias a PyVortexResult

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

break Release label indicating a breaking API change fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants