roryqi opened a new pull request, #13270:
URL: https://github.com/apache/gravitino/pull/13270

   ### What changes were proposed in this pull request?
   
   Configure the `org.casbin.jcasbin` logger at WARN by default in the
   distribution and Helm logging configurations.
   
   Add `log4j2Properties.jcasbinLoggerLevel` to allow Helm users to override
   the default, with tests covering both default and customized values.
   
   ### Why are the changes needed?
   
   JCasbin logs every authorization decision at INFO. Workloads that perform
   many authorization checks can therefore generate several megabytes of
   mostly redundant logs within seconds.
   
   Gravitino already provides authorization diagnostics at DEBUG, so these
   dependency-level decision logs should not be enabled by default.
   
   ### Does this PR introduce _any_ user-facing change?
   
   JCasbin authorization decision logs are no longer emitted at the default
   INFO level. Helm users can configure the level through
   `log4j2Properties.jcasbinLoggerLevel`.
   
   ### How was this patch tested?
   
   - `helm unittest --with-subchart=false dev/charts/gravitino`
   - `./gradlew spotlessCheck`
   - `git diff --check`
   


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

Reply via email to