Github user pvillard31 commented on the issue:

    https://github.com/apache/nifi/pull/1925
  
    @ottobackwards - you're completely right and by rebasing the PR, I missed 
something.
    Instead of using
    ````java
    grokCompiler.register(String,String);
    ````
    I should use
    ````java
    grokCompiler.register(Map<String,String>);
    ````
    That would solve the issue you're mentioning, right?
    
    The reasoning behind this PR is to remove the need of a file (deploying 
files on every node of a NiFi cluster can be annoying... and by using a 
controller service, this can be part of the flow development lifecycle since it 
can be saved in the NiFi Registry).
    
    What do you think?


---

Reply via email to