eolivelli commented on a change in pull request #1902: (WIP) Metadata checker
validating EnsemblePlacementpolicy
URL: https://github.com/apache/bookkeeper/pull/1902#discussion_r248782121
##########
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BookKeeperAdmin.java
##########
@@ -1643,4 +1643,24 @@ private static boolean
areEntriesOfSegmentStoredInTheBookie(LedgerMetadata ledge
}
return firstStoredEntryId != LedgerHandle.INVALID_ENTRY_ID;
}
+
+ /**
+ * returns boolean value specifying if the ensemble of the segment is
+ * adhering to the ensemble placement policy for the given writeQuorumSize
+ * and ackQuorumSize.
+ *
+ * @param ensembleList
+ * ensemble of the segment
+ * @param writeQuorumSize
+ * writeQuorumSize of the ledger
+ * @param ackQuorumSize
+ * ackQuorumSize of the ledger
+ * @return <tt>true</tt> if the ledger is adhering to
+ * EnsemblePlacementPolicy
+ */
+ public boolean
isEnsembleAdheringToPlacementPolicy(List<BookieSocketAddress> ensembleList, int
writeQuorumSize,
Review comment:
What about using the word 'Consistent' instead of 'Adhering' ?
Cc @jvrao
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
