You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/12/14 16:21:42 UTC

[GitHub] [accumulo-testing] EdColeman opened a new pull request #177: bump log4j version from 2.4 to 2.16.0

EdColeman opened a new pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177


   This only bumps the log4j version - unsure if additional changes are required to migrate from 2.4 --> 2.16.0


-- 
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: notifications-unsubscribe@accumulo.apache.org

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



[GitHub] [accumulo-testing] ctubbsii commented on pull request #177: bump log4j version from 2.4 to 2.16.0

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177#issuecomment-994928062


   I think this is superseded by #181. Please take a look.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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



[GitHub] [accumulo-testing] DomGarguilo commented on pull request #177: bump log4j version from 2.4 to 2.16.0

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177#issuecomment-993974088


   When i built and ran cingest off of this branch, while the shaded jar was being built the following warning got thrown:
   ```
   [WARNING] datasketches-memory-2.0.0.jar, log4j-api-2.16.0.jar define 1 overlapping classes: 
   [WARNING]   - META-INF.versions.9.module-info
   [WARNING] maven-shade-plugin has detected that some class files are
   [WARNING] present in two or more JARs. When this happens, only one
   [WARNING] single version of the class is copied to the uber jar.
   [WARNING] Usually this is not harmful and you can skip these warnings,
   [WARNING] otherwise try to manually exclude artifacts based on
   [WARNING] mvn dependency:tree -Ddetail=true and the above output.
   [WARNING] See http://maven.apache.org/plugins/maven-shade-plugin/
   ```
   I know we have fixed similar issues in the past so just wanted to document my findings.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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



[GitHub] [accumulo-testing] EdColeman commented on pull request #177: bump log4j version from 2.4 to 2.16.0

Posted by GitBox <gi...@apache.org>.
EdColeman commented on pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177#issuecomment-994940522


   If I am reading the changes from #181 correctly (which could be suspect with my git abilities today) it appears that log4j is still at 2.4
   ```
   <log4j2.version>2.4</log4j2.version>
   ```
   


-- 
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: notifications-unsubscribe@accumulo.apache.org

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



[GitHub] [accumulo-testing] EdColeman closed pull request #177: bump log4j version from 2.4 to 2.16.0

Posted by GitBox <gi...@apache.org>.
EdColeman closed pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177


   


-- 
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: notifications-unsubscribe@accumulo.apache.org

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



[GitHub] [accumulo-testing] EdColeman commented on pull request #177: bump log4j version from 2.4 to 2.16.0

Posted by GitBox <gi...@apache.org>.
EdColeman commented on pull request #177:
URL: https://github.com/apache/accumulo-testing/pull/177#issuecomment-994957608


   Superseded by #181 


-- 
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: notifications-unsubscribe@accumulo.apache.org

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