Skip to content

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Sep 22, 2025

Why are the changes needed?

Expand test coverage for Spark 4.0

How was this patch tested?

Pass GHA, also verified locally

$ build/mvn -pl :kyuubi-spark-lineage_2.13 -Pscala-2.13 -Pspark-4.0 -am clean install -DskipTests
...
$ build/mvn -pl :kyuubi-spark-lineage_2.13 -Pscala-2.13 -Pspark-4.0 test
...
Run completed in 25 seconds, 133 milliseconds.
Total number of tests run: 75
Suites: completed 5, aborted 0
Tests: succeeded 75, failed 0, canceled 0, ignored 0, pending 0
All tests passed.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  34.320 s
[INFO] Finished at: 2025-09-22T13:47:18+08:00
[INFO] ------------------------------------------------------------------------

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added kind:infra license, community building, project builds, asf infra related, etc. module:spark kind:build module:extensions labels Sep 22, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (c687c9c) to head (60a7143).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #7213   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         696     696           
  Lines       43540   43540           
  Branches     5891    5891           
======================================
  Misses      43540   43540           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pan3793 pan3793 self-assigned this Sep 22, 2025
@pan3793 pan3793 requested review from cxzl25 and wForget September 22, 2025 14:59
@pan3793 pan3793 added this to the v1.11.0 milestone Sep 22, 2025
@pan3793
Copy link
Member Author

pan3793 commented Sep 22, 2025

So currently, authZ is the only module that does not support Spark 4.0 yet

Copy link
Member

@wForget wForget left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

@pan3793 pan3793 closed this in aadf866 Sep 23, 2025
@pan3793
Copy link
Member Author

pan3793 commented Sep 23, 2025

Thanks, merged to master

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

Labels

kind:build kind:infra license, community building, project builds, asf infra related, etc. module:extensions module:spark

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants