jujoramos commented on pull request #5329:
URL: https://github.com/apache/geode/pull/5329#issuecomment-652921469


   @pivotal-eshu 
   > If we no longer need the OplogEntryIdSet after the recovery, shall we 
clear these sets once it is done?
   
   There are no references to the `OplogEntryIdSet` instance once we finish the 
recovery, so the class itself (along with all internal structures) will 
eventually be removed by the garbage collector.
   
   > With these many entryIds in drf files, it will also take longer time to 
recover. I think Darrel's idea for removing unnecessary drf files will help in 
this case as well.
   
   Agreed, I'll create another ticket for 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.

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


Reply via email to