Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate and rename abstract methods in interfaces that contain 'master' in name #4121

Conversation

xuezhou25
Copy link
Contributor

Signed-off-by: Xue Zhou [email protected]

Description

Deprecate and rename abstract methods in interface

void onMaster()
void offMaster()
TimeValue masterNodeTimeout()

Issues Resolved

#3544
#3543

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@tlfeng tlfeng changed the title Deprecate and rename abstract methods in interface Deprecate and rename abstract methods in interfaces that contain 'master' in name Aug 3, 2022
@tlfeng tlfeng added enhancement Enhancement or improvement to existing feature or request deprecate v3.0.0 Issues and PRs related to version 3.0.0 backport 2.x Backport to 2.x branch v2.2.0 backport 2.2 Backport to 2.2 branch labels Aug 3, 2022
Copy link
Collaborator

@tlfeng tlfeng left a comment

Choose a reason for hiding this comment

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

Thank you for renaming these non-inclusive public methods! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Aug 3, 2022

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #4121 (22d391f) into main (6f43dbc) will increase coverage by 0.11%.
The diff coverage is 72.72%.

@@             Coverage Diff              @@
##               main    #4121      +/-   ##
============================================
+ Coverage     70.61%   70.72%   +0.11%     
- Complexity    56983    57058      +75     
============================================
  Files          4595     4597       +2     
  Lines        274234   274240       +6     
  Branches      40181    40181              
============================================
+ Hits         193638   193946     +308     
+ Misses        64342    64016     -326     
- Partials      16254    16278      +24     
Impacted Files Coverage Δ
.../java/org/opensearch/cluster/ack/AckedRequest.java 0.00% <0.00%> (ø)
...pensearch/common/settings/IndexScopedSettings.java 100.00% <ø> (ø)
.../java/org/opensearch/snapshots/RestoreService.java 55.37% <0.00%> (ø)
...earch/cluster/LocalNodeClusterManagerListener.java 71.42% <100.00%> (ø)
...rg/opensearch/cluster/LocalNodeMasterListener.java 100.00% <100.00%> (ø)
...org/opensearch/cluster/metadata/IndexMetadata.java 84.36% <100.00%> (+0.50%) ⬆️
.../main/java/org/opensearch/index/IndexSettings.java 86.35% <100.00%> (ø)
.../java/org/opensearch/node/NodeClosedException.java 50.00% <0.00%> (-50.00%) ⬇️
...cluster/coordination/PendingClusterStateStats.java 20.00% <0.00%> (-48.00%) ⬇️
...opensearch/persistent/PersistentTasksExecutor.java 22.22% <0.00%> (-40.75%) ⬇️
... and 468 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@xuezhou25 xuezhou25 marked this pull request as ready for review August 4, 2022 01:18
@xuezhou25 xuezhou25 requested review from a team and reta as code owners August 4, 2022 01:18
@xuezhou25 xuezhou25 merged commit ba9cdcd into opensearch-project:main Aug 4, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 4, 2022
…ter' in name (#4121)

Signed-off-by: Xue Zhou <[email protected]>
(cherry picked from commit ba9cdcd)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 4, 2022
…ter' in name (#4121)

Signed-off-by: Xue Zhou <[email protected]>
(cherry picked from commit ba9cdcd)
tlfeng pushed a commit that referenced this pull request Aug 4, 2022
…ter' in name (#4121) (#4123)

Deprecate and rename abstract methods in interface
```
- in interface LocalNodeMasterListener:
void onMaster()
void offMaster()
- in interface AckedRequest:
TimeValue masterNodeTimeout()
```

Signed-off-by: Xue Zhou <[email protected]>
(cherry picked from commit ba9cdcd)
xuezhou25 added a commit that referenced this pull request Aug 4, 2022
…ter' in name (#4121) (#4122)

Signed-off-by: Xue Zhou <[email protected]>
(cherry picked from commit ba9cdcd)

Co-authored-by: Xue Zhou <[email protected]>
@xuezhou25 xuezhou25 deleted the rename_and_deprecate_abstract_methods_in_interface_LocalNodeMasterListener branch September 2, 2022 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch backport 2.2 Backport to 2.2 branch deprecate enhancement Enhancement or improvement to existing feature or request v2.2.0 v3.0.0 Issues and PRs related to version 3.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants