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

ASF GitHub Bot commented on NIFI-4615:
--------------------------------------

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

    https://github.com/apache/nifi/pull/2276#discussion_r151711473
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/dto/DtoFactory.java
 ---
    @@ -1083,13 +1083,15 @@ public ProcessorStatusDTO 
createProcessorStatusDto(final ProcessorStatus procSta
             dto.setGroupId(procStatus.getGroupId());
             dto.setName(procStatus.getName());
             dto.setStatsLastRefreshed(new Date());
    +        dto.setRunStatus(procStatus.getRunStatus().toString());
    --- End diff --
    
    I'm not sure I follow what "else" you're referring to. No change is needed 
when merging snapshots (line 384), that part is already correct. What we need 
to account for is the runStatus when merging status (line 354). We can account 
for the runStatus there using one of the two options I proposed above after 
merging the aggregate snapshot on line 363. 


> Processor status is not reported with the API
> ---------------------------------------------
>
>                 Key: NIFI-4615
>                 URL: https://issues.apache.org/jira/browse/NIFI-4615
>             Project: Apache NiFi
>          Issue Type: Bug
>          Components: Configuration Management
>    Affects Versions: 1.4.0
>            Reporter: Sébastien Bouchex Bellomié
>
> * Start a processor
> * Request its status with the Processor/get/{id] API
> Issue : ProcessorEntity.status.runStatus is always null
> Note : DtoFactory.createProcessorStatusDto do not set the processor status



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to