Github user JPercivall commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/502#discussion_r71086975
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/logback.xml
 ---
    @@ -89,7 +89,8 @@
         <logger name="org.apache.nifi.processors" level="WARN"/>
         <logger name="org.apache.nifi.processors.standard.LogAttribute" 
level="INFO"/>
         <logger 
name="org.apache.nifi.controller.repository.StandardProcessSession" 
level="WARN" />
    -    
    +
    +    <logger name="org.apache.ignite" level="WARN"/>
    --- End diff --
    
    Instead of adding this to the default logback, which would get used for 
every NiFi instance, why not add documentation to the description or Additional 
Details section stating that periodic stats are logged and this statement, 
added to the logback.xml, will suppress them. 
    
    This will get around adding a logger definition here that's specific to the 
processor but also give the user the ability to change it if needed. Besides, 
some users may like having the stats logged by default.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to