timsaucer commented on PR #1337: URL: https://github.com/apache/datafusion-python/pull/1337#issuecomment-3734632951
@nuno-faria @kosiew In my attempt to upgrade to DataFusion 52 (release candidate) I'm running into problems again with the `test_arrow_c_stream_interrupted` test. Now it's becoming a timeout issue - the 10 seconds isn't enough in the CI runner to pass. I don't want to continue arbitrarily bumping this timeout. I'm also fairly confident that in the CI test we are *not* triggering the datafusion interrupt code in `wait_for_future` because the unit test is catching a KeyboardInterrupt and not the datafusion wrapped error. I'm leaning towards marking this test as skip and opening an issue on it since manual testing shows that the stream is getting interrupted. It would be nice to deep dive into figuring out exactly what is happening in this code and to figure out how important it is to keep this test. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
