cshannon commented on PR #3957: URL: https://github.com/apache/accumulo/pull/3957#issuecomment-1817600925
> One thing I did not get to analyze was how the addition of the new mergestate impacts the deleterows code. Does the deleterows code walk through the merging and merged states? Yeah the delete rows code walks through the same state but it doesn't need to clear the marker and it doesn't need to do anything extra as it's already idempotent. The code checks on this [line](https://github.com/apache/accumulo/pull/3957/files#diff-dd97054ac43737177293a8f3a9231f05f0e4bc5eb0a3b042cad3fb8ce31b6caaR603) and then just skips to setting COMPLETE. We could just skip MERGED altogether for delete rows but figured we should just walk through all the states but either would work -- 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]
