eolivelli commented on a change in pull request #9860:
URL: https://github.com/apache/pulsar/pull/9860#discussion_r591604122
##########
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdConsume.java
##########
@@ -404,4 +427,35 @@ public void close() {
private static final Logger log =
LoggerFactory.getLogger(ConsumerSocket.class);
}
+
+ private static class DataConsistencyChecker {
+ private final ConcurrentHashMap<String, AtomicLong> messagesPerKey =
new ConcurrentHashMap<>();
Review comment:
If you do not set the failOnDeuplicateKey option you can simply run the
client and dump a summary in the end.
This way you can make more analysis.
So it is good to also have the count per key.
I am interested in keys and not in messageid.
Probably having a per message id count will be useful as well, but that is
another usecase
----------------------------------------------------------------
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:
us...@infra.apache.org
