You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by "danny0405 (via GitHub)" <gi...@apache.org> on 2023/03/11 03:24:55 UTC

[GitHub] [hudi] danny0405 commented on pull request #7955: [HUDI-5649] Unify all the loggers to slf4j

danny0405 commented on PR #7955:
URL: https://github.com/apache/hudi/pull/7955#issuecomment-1464810336

   > Few questions :
   > 
   > 1. Can you give me an example of these conflicts? (GH issues or slack threads)
   > 2. How does this affect bundling? we bundle slf4j? we need to think if anything is going to break across majjor engines, environments.
   > 3. how are all the existing [log4j.properties](http://log4j.properties/) affected? across the project
   
   1. I will ask cc @kkrugler to supply a conflict example here.
   2. We do not bundle slf4j, neither we do for log4j, user should include the jar in their chasspath
   3. the log4j.properties should not be affected because it will still invoke the log4j API underneath.


-- 
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@hudi.apache.org

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