[ 
https://issues.apache.org/jira/browse/NIFI-633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14569243#comment-14569243
 ] 

Dan Bress commented on NIFI-633:
--------------------------------

Gah... I must not have seen that.  Sorry!

I'll take another crack at it tonight.  I was trying to get 'fancy' and reuse 
the existing 
[SimpleProcessorLogger|https://github.com/apache/incubator-nifi/blob/develop/nifi/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/processor/SimpleProcessLogger.java],
 which is why I added the dependency on nifi-framework-core.  Short term, I'll 
try removing this dependency and writing a little MockProcessLogger(as you 
suggested earlier) and see where that gets us.  Long term, I agree we should 
reconsider how all these pieces fit together.

> Documentation Generation: Logger
> --------------------------------
>
>                 Key: NIFI-633
>                 URL: https://issues.apache.org/jira/browse/NIFI-633
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Core Framework
>    Affects Versions: 0.1.0
>            Reporter: Matt Gilman
>            Assignee: Dan Bress
>            Priority: Minor
>             Fix For: 0.1.1
>
>         Attachments: 0001-Squashed-commit-of-the-following.patch
>
>
> During the documentation generation, the processor is created and 
> initialized. During initialization the processor is given a mock 
> ProcessorInitializationContext which returns null when calling getLogger(). 
> If a processor attempts to log anything without verifying the logger is not 
> null, a NPE is thrown and documentation generation fails.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to