kotarot commented on pull request #4589: URL: https://github.com/apache/nifi/pull/4589#issuecomment-706068741
Thank you for looking at this change. Assume that the "Bucket" property of FetchS3Object is set to "${bucket}" and "Object Key" property is set to "${key}". If we set `bucket = "valid-bucket-name"` and `key = "" (empty string)` in Variables in the process group, after this change the flowfile will be routed to failure instead of stucking as the current implementation. If a user set the failure relationship to self-loop, changing `key = "valid-key-name"` correctly while being penalized would fix this error, and the flowfiles would be then routed to success with fetching an S3 object successfully. I think this is the same semantic as the other kinds of errors, penalizing in this case also meaningful. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org