joewitt commented on code in PR #10151:
URL: https://github.com/apache/nifi/pull/10151#discussion_r2245422179


##########
nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java:
##########
@@ -542,7 +542,11 @@ public class InvokeHTTP extends AbstractProcessor {
 
     public static final Relationship RESPONSE = new Relationship.Builder()
             .name("Response")
-            .description("Response FlowFiles transferred when receiving HTTP 
responses with a status code between 200 and 299.")
+            .description("""
+            Response FlowFiles transferred when receiving HTTP responses with 
a status code between 200 and 299.
+            Enabling [Response Generation Required] changes routing behavior 
and sends FlowFiles to the Response relationship regardless of status code 
received.
+            """
+            )

Review Comment:
   If the other relationships always show then their descriptions would need to 
reflect this conditional behavior as well.
   
   For the others the conditional language might read
   Enabling [Response Generation Required] changes routing behavior and nothing 
will route to this relationship.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to