[GitHub] [accumulo] keith-turner opened a new issue #1364: RowLock reference counts may be incorrect

[GitBox <gi...@apache.org>]

keith-turner opened a new issue #1364: RowLock reference counts may be incorrect
URL: https://github.com/apache/accumulo/issues/1364
 
 
   The [ConcurrentMap.compute](https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ConcurrentMap.html#compute-K-java.util.function.BiFunction-) function javadoc contains the following sentence
   
   ```
   The default implementation may retry these steps when multiple threads attempt updates including potentially calling the remapping function multiple times.
   ```
   
   If this is true for [ConcurrentHashMap.compute](https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ConcurrentHashMap.html#compute-K-java.util.function.BiFunction-) then the changes in #1357 may lead to incorrect reference counts in the case where the mapping functions is executed multiple times.  The javadoc for ConcurrentHashMap does not say anything about executing the function multiple times and says its atomic, leading me to believe it may not.  However, I am not 100% about the behavior.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services