ctubbsii commented on issue #3201: URL: https://github.com/apache/accumulo/issues/3201#issuecomment-1443739861
I made comments on the PR. I didn't mention log4j in my code review, but I don't think log4j is a good enough reason to stop testing these older versions that users could be using. It's very easy to patch log4j in production for those older versions. I'm sure there's lots of transitive dependency version bumps and changes, but since ZK and Hadoop are deployed by users independently from Accumulo (we don't bundle them) and its other dependencies, I think it still makes sense to test with it, so long as the tests are passing. At least, I'm not sure what harm is being done by the additional checks. -- 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]
