michaeljmarshall commented on a change in pull request #11500:
URL: https://github.com/apache/pulsar/pull/11500#discussion_r685373234
##########
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedCursorMXBean.java
##########
@@ -70,4 +70,33 @@
*/
long getPersistZookeeperErrors();
+ /**
+ * Add write data to a ledger of a cursor (in bytes).
+ *
+ * @param size Size of data written to cursor (in bytes)
+ */
+ void addWriteCursorLedgerSize(long size);
Review comment:
This method is updating two variables (`writeCursorLedgerSize` and
`writeCursorLedgerLogicalSize`). It would be helpful for future readers of this
interface to see the Javadoc explain what this method should update. Otherwise,
it would be confusing to see a `getWriteCursorLedgerLogicalSize` without an
associated `addWriteCursorLedgerLogicalSize` method.
--
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: commits-unsubscr...@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
