wchevreuil commented on pull request #3749: URL: https://github.com/apache/hbase/pull/3749#issuecomment-947489877
> So let's introduce a set method in StoreFileTracker, and then I could do a bit refactoring on MigrationStoreFileTracker to make it cleaner as the src and dst tracker does not need to be a StoreFileTrackerBase any more. Or I could do the refactoring first, to introduce a set method in StoreFileTracker, then you could make use of the method to implement this PR. > > WDYT? @wchevreuil Thanks. Yeah, I think it's better to have StoreFileTracker, MigrationStoreFileTracker and StoreFileTrackerBase refactoring on a separate PR. And I'm ok to wait for this refactoring to then update this PR here accordingly. -- 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]
