You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tez.apache.org by GitBox <gi...@apache.org> on 2021/02/12 03:09:29 UTC

[GitHub] [tez] belugabehr opened a new pull request #109: TEZ-4288: Upgrade SLF4J to 1.7.30

belugabehr opened a new pull request #109:
URL: https://github.com/apache/tez/pull/109


   


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

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



[GitHub] [tez] jteagles merged pull request #109: TEZ-4288: Upgrade SLF4J to 1.7.30

Posted by GitBox <gi...@apache.org>.
jteagles merged pull request #109:
URL: https://github.com/apache/tez/pull/109


   


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

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



[GitHub] [tez] belugabehr commented on pull request #109: TEZ-4288: Upgrade SLF4J to 1.7.30

Posted by GitBox <gi...@apache.org>.
belugabehr commented on pull request #109:
URL: https://github.com/apache/tez/pull/109#issuecomment-778324876


   @jteagles Nope. I am coming at Tez via the Hive world now that Hive+MR/Spark are dead.  Hive is currently using 1.7.30:
   
   https://github.com/apache/hive/blob/master/pom.xml#L197


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

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



[GitHub] [tez] jteagles commented on pull request #109: TEZ-4288: Upgrade SLF4J to 1.7.30

Posted by GitBox <gi...@apache.org>.
jteagles commented on pull request #109:
URL: https://github.com/apache/tez/pull/109#issuecomment-778261970


   @belugabehr, tez targets jar versions at the same version as Hadoop minimum supported version except when there is good reason (security or strong need). Currently tez has hadoop 3.1.3 as minimum supported version in master branch so there is still a mismatch of slf4j versions (1.7.10 vs 1.7.25). Do you see a strong reason to upgrade 1.7.30 over 1.7.25?


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

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



[GitHub] [tez] jteagles commented on pull request #109: TEZ-4288: Upgrade SLF4J to 1.7.30

Posted by GitBox <gi...@apache.org>.
jteagles commented on pull request #109:
URL: https://github.com/apache/tez/pull/109#issuecomment-778347066


   +1. Actually, looking at the change list between 1.7.10 and 1.7.25 and between 1.7.25 and 1.7.30. I prefer going to 1.7.30.
   Going to check this into master branch.


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

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