gaozhangmin commented on a change in pull request #10853:
URL: https://github.com/apache/pulsar/pull/10853#discussion_r648836062
##########
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java
##########
@@ -1458,6 +1458,33 @@ public PersistentOfflineTopicStats getBacklog(
return internalGetBacklog(authoritative);
}
+ @GET
+
@Path("/{tenant}/{namespace}/{topic}/messageId/{messageId}/checkLedgerExists/{checkLedgerExists}/backlogSize")
Review comment:
For partitioned-topic, We should return an error if ledger not exists
on it.
But for multi-partitions topic, We return an error only when ledger not
exists on all partitions. When calculate partitioned-topic backlog size of
this multi-partitions topic, We should't throw an error if if ledger not exists
on partitioned-topic. So We need checkLedgerExists flag to judge different
situations.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
