AliRana30 commented on PR #49105: URL: https://github.com/apache/arrow/pull/49105#issuecomment-3843144755
@raulcd You're absolutely right - the axis_order size check was redundant. Since the final return axis_order() == other.axis_order() already uses vector equality (which checks size internally), that check was unnecessary. I've removed it. The PR now only contains the essential size checks for indices() and indptr() that prevent the segfault from out-of-bounds access. -- 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]
