kevinrr888 commented on PR #4191: URL: https://github.com/apache/accumulo/pull/4191#issuecomment-1908981931
> These changes look good so far. Based on what you have done so far do you think it would be possible to break this work in multiple PRs? Yes, this is my thinking of how it could be broken up: 1) A PR changing ReadOnlyFateStore.java and the issues stemming from this (current PR) 2) A PR resolving all the issues from deleting FateTxId.java (maybe combined with first) 3) A PR changing Fate.java and the issues stemming from this 4) A PR changing Repo.java and the issues stemming from this 5) Potentially more PRs However, I'm not sure if this is the best way or if these PRs standalone would actually pass all the git checks/not break the project. Not sure how interlinked everything is. -- 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]
