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

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

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

    https://github.com/apache/nifi/pull/210#discussion_r53043617
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/service/StandardControllerServiceProvider.java
 ---
    @@ -457,6 +457,7 @@ public ControllerServiceNode 
getControllerServiceNode(final String serviceIdenti
         public Set<String> getControllerServiceIdentifiers(final Class<? 
extends ControllerService> serviceType) {
             final Set<String> identifiers = new HashSet<>();
             for (final Map.Entry<String, ControllerServiceNode> entry : 
controllerServices.entrySet()) {
    +            Class<? extends ControllerService> c = 
entry.getValue().getProxiedControllerService().getClass();
    --- End diff --
    
    I don't understand why this line was added. Was the intent to reference 'c' 
below, rather than having that expression inline?


> Refactor lifecycle code for Processors and other components
> -----------------------------------------------------------
>
>                 Key: NIFI-1464
>                 URL: https://issues.apache.org/jira/browse/NIFI-1464
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Oleg Zhurakousky
>            Assignee: Oleg Zhurakousky
>             Fix For: 0.6.0
>
>
> Similar lifecycle handling improvements that went in as part of the NIFI-1164 
> for ControllerServices, could/should be applied to other  components (e..g, 
> Processors).
> The improvements may also help to address NIFI-78 (may be without killing the 
> thread).



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

Reply via email to