kszucs commented on pull request #873: URL: https://github.com/apache/arrow-datafusion/pull/873#issuecomment-916238984
> > @kszucs I took a quick look at the physical_plan::sort_preserving_merge::tests::test_async error, it is caused by the arrow-rs upgrade. I haven't had the time to trace down on the offending arrow-rs commit yet. Will probably take a closer look at it today or tomorrow. > > I am pretty sure I saw a failure in the `sort_preserving_merge` tests related to [apache/arrow-rs#656](https://github.com/apache/arrow-rs/pull/656) Thanks Andrew for the heads up! I wonder, wouldn't it better if we would assert on record batch equality rather than its string representation? -- 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]
