Skip to content

test: do not ignore test SPARK-48037#2774

Merged
kazuyukitanimura merged 4 commits intoapache:mainfrom
kazuyukitanimura:fix-1501
Apr 6, 2026
Merged

test: do not ignore test SPARK-48037#2774
kazuyukitanimura merged 4 commits intoapache:mainfrom
kazuyukitanimura:fix-1501

Conversation

@kazuyukitanimura
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #1501

Rationale for this change

What changes are included in this PR?

How are these changes tested?

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.26%. Comparing base (f09f8af) to head (891c1f5).
⚠️ Report is 697 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2774      +/-   ##
============================================
+ Coverage     56.12%   58.26%   +2.14%     
- Complexity      976     1455     +479     
============================================
  Files           119      166      +47     
  Lines         11743    14080    +2337     
  Branches       2251     2370     +119     
============================================
+ Hits           6591     8204    +1613     
- Misses         4012     4685     +673     
- Partials       1140     1191      +51     

☔ 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.

@github-actions
Copy link
Copy Markdown

Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Feb 12, 2026
@coderfender
Copy link
Copy Markdown
Contributor

I could look into this perhaps next week if we still need to address this @kazuyukitanimura ?

@github-actions github-actions bot removed the Stale label Feb 13, 2026
@kazuyukitanimura
Copy link
Copy Markdown
Contributor Author

I could look into this perhaps next week if we still need to address this @kazuyukitanimura ?

Yes please, thank you @coderfender

@kazuyukitanimura kazuyukitanimura changed the title [WIP] fix: do not ignore test SPARK-48037 fix: [WIP] do not ignore test SPARK-48037 Apr 4, 2026
@kazuyukitanimura kazuyukitanimura changed the title fix: [WIP] do not ignore test SPARK-48037 test: do not ignore test SPARK-48037 Apr 6, 2026
@kazuyukitanimura kazuyukitanimura marked this pull request as ready for review April 6, 2026 18:47
Copy link
Copy Markdown
Contributor

@mbutrovich mbutrovich left a comment

Choose a reason for hiding this comment

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

Always good to see more Spark SQL tests running. Thanks @kazuyukitanimura!

@kazuyukitanimura kazuyukitanimura merged commit 10513e5 into apache:main Apr 6, 2026
201 of 202 checks passed
@kazuyukitanimura
Copy link
Copy Markdown
Contributor Author

Thank you @mbutrovich merged

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Github Actions get stuck for SPARK-48037: Fix SortShuffleWriter lacks shuffle write related metrics

4 participants