Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-17855 Fix typo in async client implementation

This commit is contained in:
zhangduo 2017-03-31 09:37:20 +08:00
parent d033cbb715
commit 0ec1459467
4 changed files with 24 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncMetaRegionLocator.java
View File

@ -94,11 +94,12 @@ class AsyncMetaRegionLocator {
} }
void updateCachedLocation(HRegionLocation loc, Throwable exception) { void updateCachedLocation(HRegionLocation loc, Throwable exception) {
updateCachedLoation(loc, exception, l -> metaRegionLocation.get(), newLoc -> { AsyncRegionLocator.updateCachedLocation(loc, exception, l -> metaRegionLocation.get(),
newLoc -> {
for (;;) { for (;;) {
HRegionLocation oldLoc = metaRegionLocation.get(); HRegionLocation oldLoc = metaRegionLocation.get();
if (oldLoc != null && (oldLoc.getSeqNum() > newLoc.getSeqNum() if (oldLoc != null && (oldLoc.getSeqNum() > newLoc.getSeqNum() ||
|| oldLoc.getServerName().equals(newLoc.getServerName()))) { oldLoc.getServerName().equals(newLoc.getServerName()))) {
return; return;
} }
if (metaRegionLocation.compareAndSet(oldLoc, newLoc)) { if (metaRegionLocation.compareAndSet(oldLoc, newLoc)) {

3
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncNonMetaRegionLocator.java
View File

@ -22,7 +22,6 @@ import static org.apache.hadoop.hbase.HConstants.NINES;
import static org.apache.hadoop.hbase.HConstants.ZEROES; import static org.apache.hadoop.hbase.HConstants.ZEROES;
import static org.apache.hadoop.hbase.HRegionInfo.createRegionName; import static org.apache.hadoop.hbase.HRegionInfo.createRegionName;
import static org.apache.hadoop.hbase.TableName.META_TABLE_NAME; import static org.apache.hadoop.hbase.TableName.META_TABLE_NAME;
import static org.apache.hadoop.hbase.client.AsyncRegionLocator.updateCachedLoation;
import static org.apache.hadoop.hbase.client.ConnectionUtils.createClosestRowAfter; import static org.apache.hadoop.hbase.client.ConnectionUtils.createClosestRowAfter;
import static org.apache.hadoop.hbase.client.ConnectionUtils.isEmptyStopRow; import static org.apache.hadoop.hbase.client.ConnectionUtils.isEmptyStopRow;
import static org.apache.hadoop.hbase.util.Bytes.BYTES_COMPARATOR; import static org.apache.hadoop.hbase.util.Bytes.BYTES_COMPARATOR;
@ -442,7 +441,7 @@ class AsyncNonMetaRegionLocator {
} }
void updateCachedLocation(HRegionLocation loc, Throwable exception) { void updateCachedLocation(HRegionLocation loc, Throwable exception) {
updateCachedLoation(loc, exception, l -> { AsyncRegionLocator.updateCachedLocation(loc, exception, l -> {
TableCache tableCache = cache.get(l.getRegionInfo().getTable()); TableCache tableCache = cache.get(l.getRegionInfo().getTable());
if (tableCache == null) { if (tableCache == null) {
return null; return null;

2
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncRegionLocator.java
View File

@ -98,7 +98,7 @@ class AsyncRegionLocator {
&& oldLoc.getServerName().equals(loc.getServerName()); && oldLoc.getServerName().equals(loc.getServerName());
} }
static void updateCachedLoation(HRegionLocation loc, Throwable exception, static void updateCachedLocation(HRegionLocation loc, Throwable exception,
Function<HRegionLocation, HRegionLocation> cachedLocationSupplier, Function<HRegionLocation, HRegionLocation> cachedLocationSupplier,
Consumer<HRegionLocation> addToCache, Consumer<HRegionLocation> removeFromCache) { Consumer<HRegionLocation> addToCache, Consumer<HRegionLocation> removeFromCache) {
HRegionLocation oldLoc = cachedLocationSupplier.apply(loc); HRegionLocation oldLoc = cachedLocationSupplier.apply(loc);

8
hbase-server/src/test/java/org/apache/hadoop/hbase/client/AbstractTestAsyncTableScan.java
View File

@ -237,11 +237,11 @@ public abstract class AbstractTestAsyncTableScan {
@Test @Test
public void testScanWithLimit() throws Exception { public void testScanWithLimit() throws Exception {
// testScan(1, true, 998, false, 900); // from first region to last region testScan(1, true, 998, false, 900); // from first region to last region
testScan(123, true, 234, true, 100); testScan(123, true, 234, true, 100);
// testScan(234, true, 456, false, 100); testScan(234, true, 456, false, 100);
// testScan(345, false, 567, true, 100); testScan(345, false, 567, true, 100);
// testScan(456, false, 678, false, 100); testScan(456, false, 678, false, 100);
} }
@Test @Test