alamb commented on issue #8828: URL: https://github.com/apache/arrow-rs/issues/8828#issuecomment-3527412933
> I guess the main design decision involves the ordering semantics across different union variants. Rows would sort by type id first, value second. This maintains the row format's memcmp-based sorting invariant while providing a predictable cross-type ordering I think we need to make the ordering from the row converter the same as the [`cmp` kernels](https://docs.rs/arrow/latest/arrow/compute/kernels/cmp/index.html) and https://docs.rs/arrow/latest/arrow/array/fn.make_comparator.html I took a look through the source for those kernels, and they don't seem to support UnionArray yet. So the good news is that you aren't constrained by an existing implementation -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
