accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [accumulo] keith-turner commented on issue #1254: Fix accumulo-1244 Update MetadataTableUtil to use the ample api
Date Mon, 08 Jul 2019 19:03:08 GMT
keith-turner commented on issue #1254: Fix accumulo-1244 Update MetadataTableUtil to use the
ample api
URL: https://github.com/apache/accumulo/pull/1254#issuecomment-509351579
 
 
   > why is this checked? what else could it be?
   
   Tablet information is stored in the metadata table using a row of `<table id>;<table
end row>`.  A tablets previous end row is stored as a column in the metadata table with
the row previously mentioned.
   
   The following call only uses `extent.getEndRow()` to lookup the metadata information. 
It also ask it to read the PREV_ROW column.  However, no checks are done by Ampl to ensure
that `extent.getPrevEndRow()` matches the value of the PREV_ROW column.  
   
   ```java
   context.getAmple().readTablet(extent, FILES, LOGS, PREV_ROW, DIR)
   ```
   
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message