Skip to content

[GLUTEN-11106][VL] Spark 3.5 / Delta 3.3: Add DeltaSuite / UpdateSQLSuite test suits#11041

Merged
zhztheplayer merged 2 commits intoapache:mainfrom
zhztheplayer:wip-delta-ut-3.3
Nov 12, 2025
Merged

[GLUTEN-11106][VL] Spark 3.5 / Delta 3.3: Add DeltaSuite / UpdateSQLSuite test suits#11041
zhztheplayer merged 2 commits intoapache:mainfrom
zhztheplayer:wip-delta-ut-3.3

Conversation

@zhztheplayer
Copy link
Copy Markdown
Member

@zhztheplayer zhztheplayer commented Nov 6, 2025

As title. Import more test suites from Delta 3.3 to Gluten's code base.

Moving forward, we may need a way to leverage Delta's unit tests with less code change to Gluten. Maybe by downloading the test code during compilation time, or by release a test jar by ourselves.

Related issue: #11106

As title. Import more test suites from Delta 3.3 to Gluten's code base.

Moving forward, we may need a way to leverage Delta's unit tests with less code change to Gluten. Maybe by downloading the test code during compilation time, or by release a test jar by ourselves.
@github-actions github-actions Bot added the VELOX label Nov 6, 2025
Copy link
Copy Markdown
Contributor

@jinchengchenghh jinchengchenghh left a comment

Choose a reason for hiding this comment

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

The directory name may be noticed, for iceberg, it is src-iceberg34, 34 means the Spark version34, but this src-delta33 looks like is delta version.

@jinchengchenghh
Copy link
Copy Markdown
Contributor

My PR also failed with cpp-test-udf-test (pull_request), and same failure

@zhztheplayer
Copy link
Copy Markdown
Member Author

My PR also failed with cpp-test-udf-test (pull_request), and same failure

Yes, that error seemed unrelated.

@zhztheplayer
Copy link
Copy Markdown
Member Author

The directory name may be noticed, for iceberg, it is src-iceberg34, 34 means the Spark version34

I found src-icebreg-spark34. Am I getting something wrong here?

@jinchengchenghh
Copy link
Copy Markdown
Contributor

The directory name may be noticed, for iceberg, it is src-iceberg34, 34 means the Spark version34

I found src-icebreg-spark34. Am I getting something wrong here?
OK, I might have remembered the folder name wrong.

@zhztheplayer zhztheplayer merged commit b7a6d2e into apache:main Nov 12, 2025
144 of 149 checks passed
@zhztheplayer
Copy link
Copy Markdown
Member Author

@jinchengchenghh I see. Thanks for reviewing.

@zhztheplayer zhztheplayer changed the title [VL] Spark 3.5 / Delta 3.3: Add DeltaSuite / UpdateSQLSuite test suits [GLUTEN-11106][VL] Spark 3.5 / Delta 3.3: Add DeltaSuite / UpdateSQLSuite test suits Nov 17, 2025
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.

2 participants