viirya commented on PR #7538: URL: https://github.com/apache/arrow-datafusion/pull/7538#issuecomment-1717899138
> > I am not sure if you have seen the fix for the test from @viirya in #7534 > > I've seen that PR but I don't think it's just an issue of the test. Thanks @alamb and @sarutak. Yea, as I commented in #7534 and @alamb's description in #7546, for DataFusion users this is not likely an issue except for the cases like the test that you construct physical query plan (like we did internally also) and execute it. So my idea was to propose a simplest fix to the test. As this doesn't look like a urgent bug to fix, I was thinking to find some time later to propose another fix to avoid `TaskContext` to be dropped in sort execution plan, but not long after I saw this PR was proposed. 😄 -- 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]
