zhangfengcdt commented on PR #7316: URL: https://github.com/apache/datafusion-comet/pull/7316#issuecomment-6687896118
> Reviewed [30ee2c7](https://github.com/apache/datafusion-comet/commit/30ee2c7a51527f1ce7a717bd9e26b9bc1cdecc87) across five independent scopes. The earlier volatile-sibling and range-shuffle findings are addressed, and all eight committed scalar-expansion unit tests pass. One dynamic-IN path still bypasses the bound. I confirmed it with an added native estimator test, then reproduced the query failure using freshly compiled JVM code and the exact-head native library on Spark 4.1.3. Spark and the smaller-batch native control both succeed. Current-head GitHub build/test jobs were skipped, so the validation here is local. Thanks @sunchao @rich7420! These should be fixed now. -- 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]
