vibhatha commented on code in PR #43903: URL: https://github.com/apache/arrow/pull/43903#discussion_r1741380295
########## java/memory/memory-core/src/main/java/org/apache/arrow/memory/LowCostIdentityHashMap.java: ########## @@ -317,7 +317,7 @@ public boolean isEmpty() { * @return the number of mappings in this Map. */ public int size() { - return size; Review Comment: Would it be okay to introduce a `sizeAsLong` method to be introduced and mark `size` function as deprecated. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org