Apache9 commented on pull request #3286: URL: https://github.com/apache/hbase/pull/3286#issuecomment-846034847
To align with master, we'd better just use put instead of computeIfAbsent. Notice that, in our current implementation, it is impossible for us to find out the queue is already present when calling the put method. There is only one thread to do the addition. Why we need to use ConcurrentHashMap here, is because that we may put queues for different logPrefixes at the same time. Thanks. -- 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]
