ctubbsii commented on PR #2712: URL: https://github.com/apache/accumulo/pull/2712#issuecomment-1136630928
I think this is good to merge now. I pinged @dlmarion for an updated review, but I think his comments have already been fully addressed. He can always follow up with an additional review after the fact. Thanks for working on this. It was interesting how those partially mocked objects in the tests messed with the ability to fix this easily, but I'm glad for the opportunity to fix those tests. Thanks for the PR @kitswas . If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/main/pages/people.md and leave a reference to `apache/accumulo#2712` in your commit log. If you intend to be a regular contributor to Accumulo projects, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself. :smiley_cat: -- 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]
