[GitHub] [samza] ZitingShen edited a comment on pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util

[GitBox <gi...@apache.org>]

ZitingShen edited a comment on pull request #1509:
URL: https://github.com/apache/samza/pull/1509#issuecomment-878550123


   > Seems like this isn't necessarily tied to OSS. The version in OSS is already a higher version than the one in the error log (2.12.0 > 2.11.2).
   > 
   > It seems like there are other overrides within LI that is bringing in a lower version that what is even required in the OSS in the first place. While we can independently bump the log4j2 version in OSS, I don't think the problem statement is accurate to warrant that change.
   
   @mynameborat I pasted the wrong error messages in the description :) The master and 1.6 branches are failing with 2.12.0, so the version of log4j2 on OSS does need a bump. For 1.5 branch I need more investigation as you said.
   


-- 
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: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org