youhailang removed a comment on pull request #3286: URL: https://github.com/apache/hbase/pull/3286#issuecomment-846334241
> 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. When `hbase.wal.provider` is set to multiwal and `hbase.wal.regiongrouping.numgroups` default to 2, two different logPrefixs will be put to the queues at the same time. -- 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]
