kkrugler commented on PR #6703: URL: https://github.com/apache/pinot/pull/6703#issuecomment-1115404396
@KKcorps - A jar that has a `main()` method that's used from the command line needs a logging implementation. Everything else should only be using the `slf4j-api`. Otherwise (e.g. with the Hadoop segment building workflow, which uses various Pinot jars) there can be logging implementation collisions. I made a run at fixing this up, but it got very complicated due to other 3rd party libraries used by Pinot pulling in their own logging implementations, which necessitates using various xxx-to-slf4j bridges. Right now it's kind of a mess.... -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
