Skip to content

Curator 5.7.1 Upgrade#16617

Closed
razinbouzar wants to merge 5 commits intoapache:masterfrom
razinbouzar:curator-upgrade
Closed

Curator 5.7.1 Upgrade#16617
razinbouzar wants to merge 5 commits intoapache:masterfrom
razinbouzar:curator-upgrade

Conversation

@razinbouzar
Copy link
Copy Markdown
Contributor

@razinbouzar razinbouzar commented Jun 17, 2024

Version bump of Curator from 5.3.0 to 5.7.0 due as as fix to CURATOR-696 being included in the new version.

Fixes #16411 in addition to reverting the change in #16425.

Description

This is a minor version bump of Curator.

Fixed the bug ...

Addresses the bug reported in #17781.

Renamed the class ...

Added a forbidden-apis entry ...

Release note


Key changed/added classes in this PR

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

Version bump of Curator
@razinbouzar
Copy link
Copy Markdown
Contributor Author

razinbouzar commented Jun 17, 2024

@kfaraz do you mind reviewing this version bump of Curator? @asdf2014 perhaps you can take a look as well since you submitted the original PR to revert.

@kfaraz kfaraz self-requested a review June 18, 2024 11:19
@kfaraz
Copy link
Copy Markdown
Contributor

kfaraz commented Jun 18, 2024

Thanks for keeping tabs on this, @razinbouzar !
I plan to take a look at this soon. Want to be sure if the other fix you did in #16528 would still be needed after this upgrade.

@gianm gianm added this to the 31.0.0 milestone Jul 2, 2024
@gianm
Copy link
Copy Markdown
Contributor

gianm commented Jul 2, 2024

Tagging with 31.0.0 to ensure we don't release 31.0.0 with the older Curator 5.3. We did 30.0.0 with Curator 5.5 (see #16445) and would want to release 31.0.0 with either Curator 5.5 or 5.7.

@kfaraz
Copy link
Copy Markdown
Contributor

kfaraz commented Sep 27, 2024

Removing this from Druid 31 milestone.

It had been originally included to ensure that Druid 31 was not released with Curator 5.3.
But since both master and 31.0.0 branch are already on Curator 5.5, this is not needed anymore.

As for upgrade to 5.7, it would require further testing.

@kfaraz kfaraz removed this from the 31.0.0 milestone Sep 27, 2024
@razinbouzar razinbouzar changed the title Curator 5.7.0 Upgrade Curator 5.7.1 Upgrade Mar 7, 2025
@razinbouzar
Copy link
Copy Markdown
Contributor Author

This upgrade may resolve this other bug condition reported in #17781.

Requires an update to ddsketch pom.xml and integration-texts-ex cases pom.xml.
gianm added a commit to gianm/druid that referenced this pull request Apr 1, 2025
Follow-up to apache#16617. Among other things, this version of Curator fixes
bugs related to leader election: https://issues.apache.org/jira/browse/CURATOR-696
and https://issues.apache.org/jira/browse/CURATOR-724.

Hopefully fixes apache#17781.
@gianm gianm mentioned this pull request Apr 1, 2025
gianm added a commit that referenced this pull request Apr 3, 2025
* Update to Curator 5.8.0.

Follow-up to #16617. Among other things, this version of Curator fixes
bugs related to leader election: https://issues.apache.org/jira/browse/CURATOR-696
and https://issues.apache.org/jira/browse/CURATOR-724.

Hopefully fixes #17781.

* Update licenses.yaml.

* Additional variables.
@razinbouzar
Copy link
Copy Markdown
Contributor Author

Handled in #17857.

@razinbouzar razinbouzar closed this Apr 3, 2025
@razinbouzar razinbouzar deleted the curator-upgrade branch April 3, 2025 21:51
airlock-confluentinc Bot pushed a commit to confluentinc/druid that referenced this pull request Jul 18, 2025
* Update to Curator 5.8.0.

Follow-up to apache#16617. Among other things, this version of Curator fixes
bugs related to leader election: https://issues.apache.org/jira/browse/CURATOR-696
and https://issues.apache.org/jira/browse/CURATOR-724.

Hopefully fixes apache#17781.

* Update licenses.yaml.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 Coordinators Elected Leader

3 participants