viirya commented on PR #6664: URL: https://github.com/apache/datafusion-comet/pull/6664#issuecomment-6070500247
Thanks for the benchmark numbers and the footer-read docs. Both answer what I asked. `94915161e` is a good catch: with #6693 on `main`, the new default would have sent every merge-on-read write through the split plan for no benefit. Since `run-all-spark-profiles` was removed on 10/07, the strategy change and the new merge-on-read test have only run on Spark 4.1. Could you add the label back for one more round before merge? #6740 is in now, so merging `main` should also clear the Spark 4.2 expressions failure. -- 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]
