felipepessoto commented on PR #12808: URL: https://github.com/apache/gluten/pull/12808#issuecomment-5334401655
> **Not intended to merge.** This exists to produce a CI result, alongside #12783. **Result: green.** All 8 Delta suite shards and the aggregate job passed, with Velox's per-operator output validation enabled throughout. #12783 is this same branch without the one line that applies facebookincubator/velox#18536, on the same base commit and the same pinned Velox, and 5+ of its 8 shards fail with `Output validation failed for [operator: TableScan, plan node ID: 0]`. So the upstream fix is the only difference, and it clears the whole suite -- with no regressions and no other validation failures anywhere in it. Plan from here: once velox#18536 merges it arrives through the regular weekly Velox bump, after which a separate PR removes the DV bitmap entries from `flaky-error-patterns.txt` and closes #12377. This PR and #12783 can be closed at that point. -- 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]
