[GitHub] [accumulo] keith-turner commented on issue #1453: Update Last Location. Solution for #1169

[GitBox <gi...@apache.org>]

keith-turner commented on issue #1453: Update Last Location. Solution for #1169
URL: https://github.com/apache/accumulo/pull/1453#issuecomment-594846838
 
 
   @Manno15 I don't think the change in 700ed61 will work. Need to clear the last location that is currently stored in the metadata table.  When a tablet server processes a tablet assignment it reads the metadata table and has this information.  So that info could be passed to setLocation.  The diff below shows how the tserver could pass this info.  Then need to modify the functions all the way through to where the actual metadata update is made.
   
   ```diff
   diff --git a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
   index c9a27ceff..80e780c65 100644
   --- a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
   +++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
   @@ -2508,7 +2508,7 @@ public class TabletServer extends AbstractServer {
              throw new RuntimeException("Minor compaction after recovery fails for " + extent);
            }
            Assignment assignment = new Assignment(extent, getTabletSession());
   -        TabletStateStore.setLocation(getContext(), assignment);
   +        TabletStateStore.setLocation(getContext(), assignment, tabletMetadata.getLast());
    
            synchronized (openingTablets) {
              synchronized (onlineTablets) {
   ```

----------------------------------------------------------------
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