wuzhanpeng commented on a change in pull request #13575:
URL: https://github.com/apache/pulsar/pull/13575#discussion_r792373311
##########
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java
##########
@@ -146,6 +151,13 @@
protected static final int AsyncOperationTimeoutSeconds = 30;
+ protected static final String DELETABLE_LEDGER_MARKER_KEYWORD =
"pulsar.ml.deletable.ledgers";
Review comment:
Thanks for your question. IMO, the problem of over-size znode should
theoretically be a discussion about whether `ManagedLedgerInfo` should be
stored on zk. Or we can think of it this way, the ledger information stored in
the property map actually comes from the `LedgerInfoList` structure, and this
part of the data is also stored in a znode. Therefore, if the number of ledgers
is large, the corresponding ml-znode must be huge. To avoid complicating
matters, I think we can skip this part in this proposal, and start a new
discussion thread on this problem.
--
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]
