kbendick commented on pull request #3154: URL: https://github.com/apache/iceberg/pull/3154#issuecomment-923599313
> +1 for what @rdblue says, I think just for backwards compatibility it's better to not change existing log levels. Users can always configure to ignore this in the log4j config. +1 to what both Ryan and Jack said. These are very often the logs that people quote when reporting issues. And relative to the frequency with which frameworks log (Spark, etc), the logging amount here is pretty minimal. If it's not possible to suppress these specific log lines via the Log4j configuration, possibly we could move them to come from their own class that could then be suppressed... but I think lowering their level globally is probably not the best approach. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
