felipepessoto opened a new pull request, #13112:
URL: https://github.com/apache/gluten/pull/13112
<!--
Thank you for submitting a pull request! Here are some tips:
1. For first-time contributors, please read our contributing guide:
https://github.com/apache/gluten/blob/main/CONTRIBUTING.md
2. If necessary, create a GitHub issue for discussion beforehand to avoid
duplicate work.
3. If the PR is specific to a single backend, include [VL] or [CH] in the PR
title to indicate the
Velox or ClickHouse backend, respectively.
4. If the PR is not ready for review, please mark it as a draft.
-->
## What changes are proposed in this pull request?
<!--
Provide a clear and concise description of the changes introduced in this PR.
Ensure the PR description aligns with the code changes, especially after
updates.
If applicable, include "Fixes #<GitHub_Issue_ID>" to automatically close the
corresponding issue
when the PR is merged.
-->
**Diagnostic experiment only; not a fix and not intended to merge.**
Temporarily remove `OffloadDeltaProject` and `OffloadDeltaFilter` from the
Velox Delta component, leaving operators containing Delta's `IncrementMetric`
on Spark's existing fallback path. Native Delta scan offload remains enabled,
and ClickHouse/Bolt behavior is unchanged.
The purpose is to run **Delta Spark UT (Gluten)** and measure how many known
failures start passing without `IncrementMetric` offload, providing evidence
for #9003 and the tracking information in #12743.
Keep the known-failure baseline, quarantines, and test selection unchanged.
Use the workflow's aggregate **now-passing** results, together with any
regressions and shard-completeness checks, rather than treating a lower failure
count alone as proof. A red baseline gate caused by now-passing tests is
expected for this experiment. Do not close either issue based on this draft.
## How was this patch tested?
<!--
Describe how the changes were tested, if applicable.
Include new tests to validate the functionality, if necessary.
For UI-related changes, attach screenshots to demonstrate the updates.
-->
- `git diff --check` passed; the commit changes only
`VeloxDeltaComponent.scala`.
- The changed path matches the existing Delta Spark UT pull-request trigger,
including draft PRs. The existing upstream Delta tests provide the experiment's
coverage; results are pending.
- Local Scala formatting was attempted through `dev/format-scala-code.sh
check`, but the environment has no configured JDK. Compilation and Spark tests
were not run locally.
- The repository license-header check was attempted; `dev/check.py`
references a missing `dev/license-header.py`. The existing license header is
unchanged.
## Was this patch authored or co-authored using generative AI tooling?
<!--
If generative AI tooling has been used in the process of authoring this
patch, please include the
phrase: 'Generated-by: ' followed by the name of the tool and its version.
If no, write 'No'.
Please refer to the [ASF Generative Tooling
Guidance](https://www.apache.org/legal/generative-tooling.html) for details.
-->
Generated-by: GitHub Copilot CLI 1.0.87-0 (GPT-6 Astra)
Fixes: #9003
--
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]
