dwsmith1983 commented on PR #6813: URL: https://github.com/apache/datafusion-comet/pull/6813#issuecomment-6084057751
> Could you and @dwsmith1983 agree on which PR carries the fix? @viirya #6310 has been open since September 28 with CI green on its current head, and #5613 builds on it. I'd land #6310 as it is and have #6813 follow as the change that moves both call sites onto one helper, which #6403 said the second PR would do and #6310 didn't. #6812 would then close as a duplicate of #6224. The other order works too: #6310 takes the move, and #6813 closes. -- 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]
