michaeljmarshall commented on code in PR #18003:
URL: https://github.com/apache/pulsar/pull/18003#discussion_r992611212


##########
conf/broker.conf:
##########
@@ -1014,6 +1014,8 @@ bookkeeperClientMinAvailableBookiesInIsolationGroups=
 # Enable/disable having read operations for a ledger to be sticky to a single 
bookie.
 # If this flag is enabled, the client will use one single bookie (by 
preference) to read
 # all entries for a ledger.
+# Please notice that sticky reads aren’t used unless ensemble size equals 
write quorum size for the ledger's
+# persistence policies.

Review Comment:
   Nit: the double negative might be confusing. Here is my attempt at an 
alternative wording, but mine doesn't feel completely right either.
   
   ```suggestion
   # Please notice that enabling sticky reads also requires that the ensemble 
size equals write quorum size for the ledger's
   # persistence policies.
   ```



-- 
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]

Reply via email to