milleruntime commented on pull request #2215: URL: https://github.com/apache/accumulo/pull/2215#issuecomment-1064134839
> The latest commit fixed the new merge conflicts. This PR is now more a WIP. Several changes to FateCommand has happened and this will need to be further tested to make sure everything still works. Also, the `FateCommandTest` needs further changes due to the API changes and removal of `SiteConfiguration` from FateCommand. It might make sense for the test to be moved to an IT. Thoughts on the test @milleruntime? Possibly. But I just created `FateCommandTest` to have as a unit test since we had nothing before. I have more changes with my refactor in https://github.com/apache/accumulo/pull/2475 -- 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]
