[MINOR][CI] Add new Delta known-failures caused by Velox decimal cast update - #12780
Open
felipepessoto wants to merge 1 commit into
Open
[MINOR][CI] Add new Delta known-failures caused by Velox decimal cast update#12780felipepessoto wants to merge 1 commit into
felipepessoto wants to merge 1 commit into
Conversation
… update The Delta-on-Gluten CI gate started reporting regressions (test failures not previously in known-failures.txt) after the daily Velox version bump in apache#12769, which pulled in upstream Velox commit a96d0415c "fix(spark): Enable ANSI-compliant cast from DECIMAL to DECIMAL". This changed DECIMAL cast/overflow/widening behavior. This adds the 4 newly-failing tests to known-failures.txt to unblock CI while the underlying behavior change is investigated. See apache#12779 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Delta-on-Gluten CI gate allowlist to account for newly introduced Delta Spark UT failures after the Velox daily bump (notably the DECIMAL→DECIMAL ANSI cast behavior change), unblocking the Delta Spark UT pipeline while the regression is investigated (ref: #12779).
Changes:
- Added 4 newly failing Delta Spark tests to the Delta Spark UT
known-failures.txtbaseline.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
The Delta-on-Gluten CI gate (Delta Spark UT pipeline) started reporting new regressions (test failures not previously in
known-failures.txt) after the daily Velox version bump in #12769, which pulled in upstream Velox commita96d0415c— "fix(spark): Enable ANSI-compliant cast from DECIMAL to DECIMAL". This changed DECIMAL cast/overflow/widening behavior.This PR adds the 4 newly-failing tests to
known-failures.txtto unblock CI while the underlying behavior change is investigated further.Why are the changes needed?
Mitigate #12779
Does this PR introduce any user-facing change?
No. This only updates the CI known-failures allowlist.
How was this patch tested?
N/A — CI configuration/test-allowlist change only. Verified the failing test names against the scheduled run https://github.com/apache/gluten/actions/runs/31866072615#summary-94980767356:
Generated-by: Claude claude-sonnet-5 (GitHub Copilot CLI)