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

    https://github.com/apache/nifi/pull/2601#discussion_r183461688
  
    --- Diff: 
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-processors/src/main/java/org/apache/nifi/processors/elasticsearch/QueryElasticsearchHttp.java
 ---
    @@ -267,30 +267,28 @@ public void setup(ProcessContext context) {
     
         @Override
         public void onPropertyModified(final PropertyDescriptor descriptor, 
final String oldValue, final String newValue) {
    +        final Set<Relationship> routeQueryInfoRels = new HashSet<>();
    +        routeQueryInfoRels.add(REL_SUCCESS);
    +        routeQueryInfoRels.add(REL_FAILURE);
    +        routeQueryInfoRels.add(REL_RETRY);
    +
    +        final Set<Relationship> successRels = new HashSet<>();
    --- End diff --
    
    I thought the same thing.  Thanks for the feedback my latest did take that 
approach


---

Reply via email to