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.
---
