iemejia commented on pull request #13799: URL: https://github.com/apache/beam/pull/13799#issuecomment-767693059
> Well, I don't know why specific Log4j version was used for testing (it was added even before my code refactoring) but if all tests pass then, seems, we don't need that anymore. So, it LGTM in general for me. > > Maybe @iemejia recalls something on this. No idea but in any case even if used it should have not had that scope but `testRuntimeOnly` so good to remove save if some test is not happy ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
