[jira] [Commented] (NIFI-1216) NiFi framework spends resources formatting log messages for Processor Loggers even when log level disabled

2015-11-25 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15028236#comment-15028236 ] ASF subversion and git services commented on NIFI-1216: --- Commit

[jira] [Commented] (NIFI-1216) NiFi framework spends resources formatting log messages for Processor Loggers even when log level disabled

2015-11-25 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15028233#comment-15028233 ] Joseph Witt commented on NIFI-1216: --- +1 . contrib check/full build/tested. good to go. Am merging >

[jira] [Commented] (NIFI-1216) NiFi framework spends resources formatting log messages for Processor Loggers even when log level disabled

2015-11-25 Thread Oleg Zhurakousky (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15026861#comment-15026861 ] Oleg Zhurakousky commented on NIFI-1216: Minor stylistic comment; Personally I'd go for 'if (true)

[jira] [Commented] (NIFI-1216) NiFi framework spends resources formatting log messages for Processor Loggers even when log level disabled

2015-11-24 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15025070#comment-15025070 ] Mark Payne commented on NIFI-1216: -- replaced patch because i forgot to add one of the classes to git

[jira] [Commented] (NIFI-1216) NiFi framework spends resources formatting log messages for Processor Loggers even when log level disabled

2015-11-24 Thread Bryan Bende (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15025121#comment-15025121 ] Bryan Bende commented on NIFI-1216: --- +1 changes look good, passes contrib-check > NiFi framework spends