felipepessoto commented on PR #13134: URL: https://github.com/apache/gluten/pull/13134#issuecomment-6181360598
@malinjawi Thanks for the review. All five inline comments are addressed in `fb6868738f`, and the branch is rebased onto current main (`b11eaa78e8`). Both observer suites now run with the `ExtendedSQLTest` selector: all nine tests passed locally on Spark 3.5 / Delta 3.3 and Spark 4.0 / Delta 4.0. The revised partial DELETE regression also fails against the original observer implementation and passes with the fix. Filed the separate footer-metadata follow-up as apache/gluten#13288 and kept that test in the baseline. After rebasing, the same 64 removals reduce the current-main baseline from 619 to 555. Fresh CI is running, including [Delta Spark UT](https://github.com/apache/gluten/actions/runs/38658839475). The PR description has been updated to reflect the revised test layout and current validation status. -- 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]
