DomGarguilo opened a new pull request, #5674:
URL: https://github.com/apache/accumulo/pull/5674

   Closes #5650 
   
   In #5652, a periodic task was added to remove any scan refs that were not 
properly removed after a scan due to an interruption. In this PR, we change 
things so that we don't even try to remove these refs (was being done in 
`returnFilesForScan()`) and instead just add them to a collection 
(`filesToDeleteAfterScan`) to be batch removed periodically. I renamed the 
methods from `removeOrphanedScanRefs()` to `removeBatchedScanRefs()` to more 
closely align with this behavior. 
   
   Additional changes were made in the Tablet code to properly mark things as a 
potential write in progress.
   
   These changes were suggested in 
https://github.com/apache/accumulo/pull/5652#pullrequestreview-2940263926 and  
https://github.com/apache/accumulo/pull/5652#discussion_r2155382782.
   
   


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

Reply via email to