Skip to content

Rename ClusterBlocks.hasGlobalBlock methods#36941

Merged
tlrx merged 1 commit intoelastic:masterfrom
tlrx:rename-cluster-blocks-has-block-methods
Jan 7, 2019
Merged

Rename ClusterBlocks.hasGlobalBlock methods#36941
tlrx merged 1 commit intoelastic:masterfrom
tlrx:rename-cluster-blocks-has-block-methods

Conversation

@tlrx
Copy link
Copy Markdown
Member

@tlrx tlrx commented Dec 21, 2018

As suggested in #36775 (comment), this pull request renames the following methods:

ClusterBlocks.hasGlobalBlock(int)
ClusterBlocks.hasGlobalBlock(RestStatus)
ClusterBlocks.hasGlobalBlock(ClusterBlockLevel)

to something that better reflects the property of the ClusterBlock that is searched for:

ClusterBlocks.hasGlobalBlockWithId(int)
ClusterBlocks.hasGlobalBlockWithStatus(RestStatus)
ClusterBlocks.hasGlobalBlockWithLevel(ClusterBlockLevel)

@tlrx tlrx added v7.0.0 :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >refactoring v6.7.0 labels Dec 21, 2018
@tlrx tlrx requested a review from ywelsch December 21, 2018 15:21
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

@tlrx tlrx merged commit 6347461 into elastic:master Jan 7, 2019
@tlrx tlrx deleted the rename-cluster-blocks-has-block-methods branch January 7, 2019 08:42
@tlrx
Copy link
Copy Markdown
Member Author

tlrx commented Jan 7, 2019

thanks @ywelsch

tlrx added a commit that referenced this pull request Jan 7, 2019
As suggested in #36775, this pull request renames the following methods:

ClusterBlocks.hasGlobalBlock(int)
ClusterBlocks.hasGlobalBlock(RestStatus)
ClusterBlocks.hasGlobalBlock(ClusterBlockLevel)

to something that better reflects the property of the ClusterBlock that is searched for:

ClusterBlocks.hasGlobalBlockWithId(int)
ClusterBlocks.hasGlobalBlockWithStatus(RestStatus)
ClusterBlocks.hasGlobalBlockWithLevel(ClusterBlockLevel)
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >refactoring v6.7.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants