[GitHub] [zookeeper] li4wang commented on pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
li4wang commented on pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802#issuecomment-1065398600 Thanks for looking into this @eolivelli For the Vulnerability issue reported from slf4j-log4j12, how about updating slf4j version from `1.7.30` to `1.7.35`? Another

[GitHub] [zookeeper] sonatype-lift[bot] commented on a change in pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802#discussion_r824757853 ## File path: zookeeper-assembly/pom.xml ## @@ -64,6 +64,14 @@ zookeeper ${project.version} + Review comment:

[GitHub] [zookeeper] eolivelli commented on pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
eolivelli commented on pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802#issuecomment-1065140139 I have rebased onto branch-3.7 after cherry-picking https://issues.apache.org/jira/browse/ZOOKEEPER-4479 (to fix C client test flakyness) -- This is an automated

[GitHub] [zookeeper] eolivelli opened a new pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
eolivelli opened a new pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802 - Drop Apache Log4j1 - Replace with https://reload4j.qos.ch/, that is 100% compatible (same package names, it is actually a fork) - Remove logging implementation dependency from zookeeper

[GitHub] [zookeeper] eolivelli commented on pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
eolivelli commented on pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802#issuecomment-1064881425 I don't know why GH does not allow me to restart CI, trying to close and reopen -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zookeeper] eolivelli closed pull request #1802: ZOOKEEPER-4455: Move to https://reload4j.qos.ch/ (remove log4j1)

2022-03-11 Thread GitBox
eolivelli closed pull request #1802: URL: https://github.com/apache/zookeeper/pull/1802 -- 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:

[GitHub] [zookeeper] maoling commented on pull request #1822: ZOOKEEPER-4474: Drop unused ZooDefs.opNames

2022-03-11 Thread GitBox
maoling commented on pull request #1822: URL: https://github.com/apache/zookeeper/pull/1822#issuecomment-1065830922 @kezhuw Thanks for your contribution. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [zookeeper] asfgit closed pull request #1822: ZOOKEEPER-4474: Drop unused ZooDefs.opNames

2022-03-11 Thread GitBox
asfgit closed pull request #1822: URL: https://github.com/apache/zookeeper/pull/1822 -- 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: