Yi Zheng created HBASE-20574:
--------------------------------
Summary: prevRowLock seems not working
Key: HBASE-20574
URL: https://issues.apache.org/jira/browse/HBASE-20574
Project: HBase
Issue Type: Improvement
Components: regionserver
Reporter: Yi Zheng
In HRegion.java, getRowLockInternal using prevRowLock to check whether this
lock is obtained before, but it may not work as imagine.
Because this parameter passed as the last lock the thread obtained, and the
code always be like this:
for (byte[] rows: rowsToLock) {
rowLock = getRowLockInternal(row, prevRowLock);
if (rowLock != prevRowLock) {
acquiredRowLocks.add(rowLock);
prevRowLock = rowLock;
}
}
prevRowLock will never equal to the rowLockContext's lock in
getRowLockInternal. I think this parameter is redundant.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)