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

ASF GitHub Bot commented on MINIFICPP-501:
------------------------------------------

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

    https://github.com/apache/nifi-minifi-cpp/pull/333#discussion_r189747427
  
    --- Diff: libminifi/include/core/state/nodes/AgentInformation.h ---
    @@ -297,9 +297,20 @@ class ComponentManifest : public DeviceInformation {
                 descriptorRequired.name = "required";
                 descriptorRequired.value = prop.second.getRequired();
     
    +            SerializedResponseNode descriptorDependentProperties;
    +            descriptorDependentProperties.name = "dependentProperties";
    +
    +            for (const auto &propName : 
prop.second.getDependentProperties()) {
    +              SerializedResponseNode descriptorDependentProperty;
    +              descriptorDependentProperty.name = propName;
    +              descriptorDependentProperty.value = propName;
    --- End diff --
    
    why is the value the property name? You don't need a value if you just need 
a list. 


> Incorporate dependent property metadata into agent information
> --------------------------------------------------------------
>
>                 Key: MINIFICPP-501
>                 URL: https://issues.apache.org/jira/browse/MINIFICPP-501
>             Project: NiFi MiNiFi C++
>          Issue Type: Improvement
>            Reporter: Andrew Christianson
>            Assignee: Andrew Christianson
>            Priority: Major
>
> Report which properties a property is dependent upon.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to