kylebarron commented on PR #7247: URL: https://github.com/apache/arrow-rs/pull/7247#issuecomment-2892178773
IMO I'd have considered this to be a breaking change, as the `StructArray::new` constructor now panics with input considered valid as of `55.0`. In [pyo3-arrow](https://docs.rs/pyo3-arrow/latest/pyo3_arrow/) I have a test that checks I can import an empty `RecordBatch` which goes through [this code path](https://github.com/kylebarron/arro3/blob/b214c74ede440fd1f1cdd03e603b2ec33ae8fff1/pyo3-arrow/src/ffi/from_python/utils.rs#L94-L106), which uses `StructArray::new` and now panics. Ideally I'd like to declare my dependency at `55.0` to let downstream users use either `55.0` or `55.1`, but I can't use the suggested workarounds of `try_new_with_length` because that didn't exist in `55`. I figure I'll just bump my requirement to `55.1` and that'll be the least painful way to handle this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
