accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] keith-turner commented on a change in pull request #797: Improved tablet metadata abstraction layer and used it more.
Date Wed, 05 Dec 2018 20:29:17 GMT
keith-turner commented on a change in pull request #797: Improved tablet metadata abstraction
layer and used it more.
URL: https://github.com/apache/accumulo/pull/797#discussion_r239225831
 
 

 ##########
 File path: core/src/main/java/org/apache/accumulo/core/clientImpl/ConcurrentKeyExtentCache.java
 ##########
 @@ -85,8 +85,8 @@ protected void updateCache(KeyExtent e) {
   @VisibleForTesting
   protected Stream<KeyExtent> lookupExtents(Text row)
       throws TableNotFoundException, AccumuloException, AccumuloSecurityException {
-    return MetadataScanner.builder().from(ctx).scanMetadataTable().overRange(tableId, row,
null)
-        .checkConsistency().fetchPrev().build().stream().limit(100).map(TabletMetadata::getExtent);
+    return TabletsMetadata.builder().forTable(tableId).overlapping(row, null).checkConsistency()
+        .fetchPrev().build(ctx).stream().limit(100).map(TabletMetadata::getExtent);
 
 Review comment:
   I will fox that separately because it needs to go into 1.9 and this change already has
too much.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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

Mime
View raw message