tustvold commented on PR #5180: URL: https://github.com/apache/arrow-rs/pull/5180#issuecomment-1845397399
Whilst I appreciate you working on this, I feel I wasn't clear enough before: * We cannot change the ordering relation of these kernels to a partial order, as it must match that of sorts, which must be a total order * We cannot add new array types that aren't part of the arrow standard From my perspective this seems like it is trying to frame range analysis as an ordering relation, which seems somewhat contrived? Perhaps this logic could be implemented within the range analysis framework of DataFusion, as opposed to here? -- 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]
