Skip to content

Upgrade Helix version to 1.3.2#16239

Merged
somandal merged 1 commit into
apache:masterfrom
somandal:helix-1.3.2-upgrade
Jun 30, 2025
Merged

Upgrade Helix version to 1.3.2#16239
somandal merged 1 commit into
apache:masterfrom
somandal:helix-1.3.2-upgrade

Conversation

@somandal

@somandal somandal commented Jun 30, 2025

Copy link
Copy Markdown
Contributor

Upgrade the Helix version to 1.3.2

Helix 1.3.2 branch: https://github.com/apache/helix/tree/helix-1.3.2
Helix 1.3.2 commits: https://github.com/apache/helix/commits/helix-1.3.2/

The main motivation for making this update is to pull in this fix from the Helix side: apache/helix#2976

Today we see an issue with Minions where if the Minion is not gracefully shutdown (even if the tag is removed), Helix can drop the tasks assigned to that minion. With the above changes, Helix will wait for the tasks to complete.

Testing done:

  • Ran the build locally and validated that the build passes: mvn clean install -DskipTests -Pbin-dist
  • Validated in the IDE that the correct 1.3.2 jar is used for the Helix related classes

@codecov-commenter

codecov-commenter commented Jun 30, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.18%. Comparing base (1a476de) to head (beb9e90).
Report is 372 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #16239      +/-   ##
============================================
+ Coverage     62.90%   63.18%   +0.28%     
+ Complexity     1386     1362      -24     
============================================
  Files          2867     2959      +92     
  Lines        163354   170860    +7506     
  Branches      24952    26151    +1199     
============================================
+ Hits         102755   107966    +5211     
- Misses        52847    54716    +1869     
- Partials       7752     8178     +426     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.17% <ø> (+0.29%) ⬆️
java-21 63.15% <ø> (+0.32%) ⬆️
skip-bytebuffers-false ?
skip-bytebuffers-true ?
temurin 63.18% <ø> (+0.28%) ⬆️
unittests 63.18% <ø> (+0.28%) ⬆️
unittests1 64.72% <ø> (+8.90%) ⬆️
unittests2 33.44% <ø> (-0.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@somandal somandal added the dependencies Pull requests that update a dependency file label Jun 30, 2025
@somandal somandal merged commit 0c6bafc into apache:master Jun 30, 2025
20 checks passed
@somandal somandal deleted the helix-1.3.2-upgrade branch June 30, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants