keith-turner commented on issue #3845:
URL: https://github.com/apache/accumulo/issues/3845#issuecomment-1764948775

   @cshannon  I looked around in the code and suspect the following strategy 
may work to fix this.
   
    * MergeStats tracks the number of wals seen
    * MergeStats.nextMergeState() does not transition out of 
MergeState.WAITING_FOR_OFFLINE if the wal count >0 for any tablet
    * Manager.getGoalState() will return hosted when a tablet has wals even if 
merge state is WAITING_FOR_OFFLINE.  May be able to get walog count from the 
passed in TabletLocationState.
    * When actually merging tablets, if wals are seen then throw an exception 
as this is not expected (should not get that far with wals).
   
   Was looking at the code in main when making the above comments.  Hoping 
WAITING_FOR_OFFLINE is common code for both merge and delete rows, did not 
check that.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to