[GitHub] [nifi] ChrisSamo632 commented on pull request #4761: NIFI-7753 enable INFO logging for ExecuteGroovyScript by default

[GitBox <gi...@apache.org>]

ChrisSamo632 commented on pull request #4761:
URL: https://github.com/apache/nifi/pull/4761#issuecomment-781588348


   > @ChrisSamo632 yeah i think i'd be in favor of simply documenting the fact that the default log level is set to WARN, and a simple explanation that the default log level used to be INFO and for many flows, just a single INFO-level log message per FlowFile per Processor resulted in tens or hundreds of GB of logs per day, and data provenance generally provides all of the same info as INFO-level logs but in a much more accessible way.
   
   Makes sense. This PR would only enable INFO for the script components, but even then there could be flows where that spits out a lot of logs.
   
   I'll close this PR and add a comment to the jira ticket, not sure where would be best to document the default logging levels (not something to do per-processor, so maybe the User Guide or Wiki)


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

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