Skip to content

[MINOR][CI] Add new Delta known-failures caused by Velox decimal cast update - #12780

Open
felipepessoto wants to merge 1 commit into
apache:mainfrom
felipepessoto:delta-known-failures-decimal-cast
Open

[MINOR][CI] Add new Delta known-failures caused by Velox decimal cast update#12780
felipepessoto wants to merge 1 commit into
apache:mainfrom
felipepessoto:delta-known-failures-decimal-cast

Conversation

@felipepessoto

@felipepessoto felipepessoto commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

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 commit a96d0415c — "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.txt to unblock CI while the underlying behavior change is investigated further.

org.apache.spark.sql.delta.ImplicitMergeCastingSuite#MERGE overflow in WHEN MATCHED THEN UPDATE SET t.value = s.value sourceType: DECIMAL(3,1), targetType: DECIMAL(3,2) followAnsiEnabled: false, ansiEnabled: true, storeAssignmentPolicy: LEGACY
org.apache.spark.sql.delta.ImplicitStreamingMergeCastingSuite#Streaming MERGE overflow sourceType: DECIMAL(3,1), targetType: DECIMAL(3,2) followAnsiEnabled: false, ansiEnabled: true, storeAssignmentPolicy: LEGACY
org.apache.spark.sql.delta.typewidening.TypeWideningInsertSchemaEvolutionBasicSuite#INSERT - never automatic type widening DECIMAL(9,2) -> DECIMAL(18,2)
org.apache.spark.sql.delta.typewidening.TypeWideningInsertSchemaEvolutionBasicSuite#INSERT - never automatic type widening DECIMAL(9,2) -> DECIMAL(19,3)

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)

… 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>
Copilot AI lite review requested due to automatic review settings August 15, 2026 01:26
@github-actions github-actions Bot added the INFRA label Aug 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.txt baseline.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants