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

    https://github.com/apache/nifi/pull/3122#discussion_r230650492
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestLogMessage.java
 ---
    @@ -57,7 +57,7 @@ MockComponentLog getMockComponentLog() {
         public void before() throws InitializationException {
             testableLogMessage = new TestableLogMessage();
             runner = TestRunners.newTestRunner(testableLogMessage);
    -
    +        runner.setValidateExpressionUsage(false);
    --- End diff --
    
    @pvillard31 @ijokarumawak Thank you for your reviews.
    
    At first, the purpose of my PR was, to enable to pick up a Log Level from 
the pull-down list (as we can do it on the LogAttribute processor), so that I 
would like to make it easier for users (including NiFi beginners) to use this 
processor.
    
    As @ijokarumawak has suggested, should I fix it to be EL acceptable for 
flexible logging usages?


---

Reply via email to