kitswas commented on PR #2712: URL: https://github.com/apache/accumulo/pull/2712#issuecomment-1136671487
> 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. In retrospect, commit 6189662b0766debf42d199e148e8f52a9c6fe13c and the tests came close to the scenario described in the [Broken windows theory](https://blog.codinghorror.com/the-broken-window-theory/). If commit 6189662b0766debf42d199e148e8f52a9c6fe13c were kept, it would have been an excuse to skip fixing the tests in future. > 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. 😺 Very tempting, almost dangerously so, but I think I will pass. My passion (programming, computers) is different from my profession. It has always been so. Hopefully, they will merge after I graduate. I primarily use Windows. [Apache Accumulo does not like that](https://stackoverflow.com/a/15540833/8659747). While I can [build using WSL](https://stackoverflow.com/a/72335829/8659747), WSL does not mesh well with most GUI tools (IntelliJ, Gitkraken, ...) other than VS Code. > 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. Is it alright to do this when you did most of the work? -- 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]
