You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "DomGarguilo (via GitHub)" <gi...@apache.org> on 2023/02/24 21:50:54 UTC

[GitHub] [accumulo] DomGarguilo opened a new pull request, #3204: Remove log4j-core dependency

DomGarguilo opened a new pull request, #3204:
URL: https://github.com/apache/accumulo/pull/3204

   In #3000 log4j-core dependency was added. It needed to be there due to https://issues.apache.org/jira/browse/LOG4J2-3601. This has been fixed as of version `2.20.0`. This PR updates to that version and removes the changes that were needed before the patch.


-- 
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] cshannon commented on pull request #3204: Remove log4j-core dependency

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on PR #3204:
URL: https://github.com/apache/accumulo/pull/3204#issuecomment-1445123939

   I think we should probably target this for 2.1 as well since that is a LTM version so it should also get dependency updates.


-- 
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] ctubbsii commented on pull request #3204: Remove log4j-core dependency

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on PR #3204:
URL: https://github.com/apache/accumulo/pull/3204#issuecomment-1453596413

   Superseded by #3215 to apply this to 2.1


-- 
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] ctubbsii closed pull request #3204: Remove log4j-core dependency

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii closed pull request #3204: Remove log4j-core dependency
URL: https://github.com/apache/accumulo/pull/3204


-- 
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] DomGarguilo commented on pull request #3204: Remove log4j-core dependency

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #3204:
URL: https://github.com/apache/accumulo/pull/3204#issuecomment-1446729381

   > I think we should probably target this for 2.1 as well since that is a LTM version so it should also get dependency updates.
   
   Created #3215 tracking 2.1 branch @cshannon 


-- 
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