Internal: Make TransportMasterNodeOperationAction#checkBlock abstract
#7740
Assignees
Comments
kimchy
added a commit
that referenced
this issue
Sep 17, 2014
Master node related operations were missing proper handling of cluster blocks, allowing for example to perform cluster level update settings even before the state was fully restored on initial cluster startup Note, the change allows to change read only related settings without checking for blocks on update settings, as without it, it means one can't re-enable metadata/write. Also, it doesn't check for blocks on cluster state and health API, as those are allowed to be used even when blocked to figure out what causes the block. closes #7763 closes #7740
kimchy
added a commit
that referenced
this issue
Sep 17, 2014
Master node related operations were missing proper handling of cluster blocks, allowing for example to perform cluster level update settings even before the state was fully restored on initial cluster startup Note, the change allows to change read only related settings without checking for blocks on update settings, as without it, it means one can't re-enable metadata/write. Also, it doesn't check for blocks on cluster state and health API, as those are allowed to be used even when blocked to figure out what causes the block. closes #7763 closes #7740
clintongormley
changed the title
TransportMasterNodeOperationAction#checkBlock abstractTransportMasterNodeOperationAction#checkBlock abstract
mute
pushed a commit
to mute/elasticsearch
that referenced
this issue
Jul 29, 2015
Master node related operations were missing proper handling of cluster blocks, allowing for example to perform cluster level update settings even before the state was fully restored on initial cluster startup Note, the change allows to change read only related settings without checking for blocks on update settings, as without it, it means one can't re-enable metadata/write. Also, it doesn't check for blocks on cluster state and health API, as those are allowed to be used even when blocked to figure out what causes the block. closes elastic#7763 closes elastic#7740
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
TransportMasterNodeOperationAction#checkBlockshould be implemented by any subclasses of ``TransportMasterNodeOperationActionbut it's returningnull` by default. We should make that abstract to force implementations for it.The text was updated successfully, but these errors were encountered: