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

    https://github.com/apache/nifi/pull/3051#discussion_r237138678
  
    --- Diff: 
nifi-nar-bundles/nifi-cassandra-bundle/nifi-cassandra-processors/src/main/java/org/apache/nifi/processors/cassandra/AbstractCassandraProcessor.java
 ---
    @@ -132,17 +134,25 @@
                 .addValidator(StandardValidators.CHARACTER_SET_VALIDATOR)
                 .build();
     
    +    // Relationships
         public static final Relationship REL_SUCCESS = new 
Relationship.Builder()
                 .name("success")
                 .description("A FlowFile is transferred to this relationship 
if the operation completed successfully.")
                 .build();
    +
    +    static final Relationship REL_ORIGINAL = new Relationship.Builder()
    +            .name("original")
    +            .description("All input FlowFiles that are part of a 
successful query execution go here.")
    +            .build();
    +
         public static final Relationship REL_FAILURE = new 
Relationship.Builder()
                 .name("failure")
    -            .description("A FlowFile is transferred to this relationship 
if the operation failed.")
    +            .description("CQL query execution failed.")
                 .build();
    +
         public static final Relationship REL_RETRY = new 
Relationship.Builder().name("retry")
    -            .description("A FlowFile is transferred to this relationship 
if the operation cannot be completed but attempting "
    -                    + "it again may succeed.")
    +            .description("A FlowFile is transferred to this relationship 
if the query cannot be completed but attempting "
    --- End diff --
    
    I tend to like `operation` here (see above), and I like that you replaced 
`it` with `operation` for clarity.


---

Reply via email to