[ https://issues.apache.org/jira/browse/KAFKA-5967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16184797#comment-16184797 ]
Matthias J. Sax commented on KAFKA-5967: ---------------------------------------- [~sssanthalingam] Please send an email with you user name to Kafka dev list and ask to be added to contributors list. Than you can assign to yourself. > Ineffective check of negative value in > CompositeReadOnlyKeyValueStore#approximateNumEntries() > --------------------------------------------------------------------------------------------- > > Key: KAFKA-5967 > URL: https://issues.apache.org/jira/browse/KAFKA-5967 > Project: Kafka > Issue Type: Bug > Reporter: Ted Yu > Priority: Minor > Labels: beginner, newbie > > {code} > long total = 0; > for (ReadOnlyKeyValueStore<K, V> store : stores) { > total += store.approximateNumEntries(); > } > return total < 0 ? Long.MAX_VALUE : total; > {code} > The check for negative value seems to account for wrapping. > However, wrapping can happen within the for loop. So the check should be > performed inside the loop. -- This message was sent by Atlassian JIRA (v6.4.14#64029)