rdhabalia commented on a change in pull request #2041: Added ReadOnlyCursor to
ManagedLedger
URL: https://github.com/apache/incubator-pulsar/pull/2041#discussion_r199384543
##########
File path:
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerFactoryImpl.java
##########
@@ -255,6 +260,64 @@ public void initializeFailed(ManagedLedgerException e) {
});
}
+
+
+ @Override
+ public ReadOnlyCursor openReadOnlyCursor(String managedLedgerName,
Position startPosition, ManagedLedgerConfig config)
+ throws InterruptedException, ManagedLedgerException {
+ class Result {
Review comment:
is there any reason defining class in method and not outside of method?
----------------------------------------------------------------
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
