ifesdjeen commented on code in PR #2807:
URL: https://github.com/apache/cassandra/pull/2807#discussion_r1363391055
##########
src/java/org/apache/cassandra/tcm/ClusterMetadataService.java:
##########
@@ -125,6 +125,10 @@ public static ClusterMetadataService instance()
private final AtomicBoolean commitsPaused = new AtomicBoolean();
+ /**
+ * Returns the state of the {@code ClusteMetadataService}.
Review Comment:
This comment seems to be tautological. Maybe add "returns _current_ state"
at least? I would say it could be useful to specify what gets returned, too. Ie
whether or not current cluster metadata service is local (ie member of CMS),
remote (ie a non-CMS member of the cluster), gossip (ie a node that has not yet
been upgraded to TCM), or the node is forced to reset the local metadata.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
