kevinjqliu commented on PR #16725: URL: https://github.com/apache/iceberg/pull/16725#issuecomment-4813046551
Thanks for working on this! I pushed a small adjustment to the approach. The previous version baked a `simplelogger.properties` file into the image and used `LOG_LEVEL` to override the default level. The updated version avoids shipping a fixed properties file and instead adds a small `entrypoint.sh`. With the new approach: - `LOG_LEVEL=WARN` still works as the simple way to reduce logs. - Users can also mount their own `simplelogger.properties` and pass its directory with `LOG_CONFIG_DIR`. - If both are set, `LOG_LEVEL` takes precedence for the default log level, while the properties file can still configure other `slf4j-simple` settings. This keeps the default behavior the same, but gives users a more flexible way to provide full logging configuration without rebuilding the image. -- 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]
