You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by GitBox <gi...@apache.org> on 2021/07/12 20:04:22 UTC
[GitHub] [samza] ZitingShen opened a new pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
ZitingShen opened a new pull request #1509:
URL: https://github.com/apache/samza/pull/1509
--
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
[GitHub] [samza] ZitingShen closed pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
ZitingShen closed pull request #1509:
URL: https://github.com/apache/samza/pull/1509
--
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
[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
Posted by 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
[GitHub] [samza] ZitingShen removed a comment on pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
ZitingShen removed a comment on pull request #1509:
URL: https://github.com/apache/samza/pull/1509#issuecomment-878548024
> 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.
--
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
[GitHub] [samza] ZitingShen commented on pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
ZitingShen commented on pull request #1509:
URL: https://github.com/apache/samza/pull/1509#issuecomment-878614084
The root cause was found in samza-release-tests. Close the PR.
--
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
[GitHub] [samza] ZitingShen closed pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
ZitingShen closed pull request #1509:
URL: https://github.com/apache/samza/pull/1509
--
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
[GitHub] [samza] ZitingShen commented on pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
ZitingShen commented on pull request #1509:
URL: https://github.com/apache/samza/pull/1509#issuecomment-878548024
--
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
[GitHub] [samza] mynameborat commented on pull request #1509: Bump log4j2 version to 2.12.1 to be compatible with latest container and util
Posted by GitBox <gi...@apache.org>.
mynameborat commented on pull request #1509:
URL: https://github.com/apache/samza/pull/1509#issuecomment-878517567
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.
--
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