cshannon commented on PR #3975: URL: https://github.com/apache/accumulo/pull/3975#issuecomment-1821825332
@keith-turner - I put the code to clean up the `MERGED` marker in `DeleteTablets` because this seemed like the simplest place to put it. I could also move it into `FinishTableRangeOp` but that currently can be called even if there is no marker to clean up (such as if only 1 tablet for the merge range and it skips) where as `DeleteTablets` should only be called if there is a `MERGED` marker to clean up. But I can move the clean up there if you think that makes more sense and just check if the marker exists before removing it or just remove it regardless (I don't think there's any harm in deleting a column that doesn't exist) -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
