pkelly-nifi commented on a change in pull request #4576:
URL: https://github.com/apache/nifi/pull/4576#discussion_r506560739
##########
File path:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/s3/FetchS3Object.java
##########
@@ -147,6 +171,11 @@ public void onTrigger(final ProcessContext context, final
ProcessSession session
request = new GetObjectRequest(bucket, key, versionId);
}
request.setRequesterPays(requesterPays);
+ if(rangeLength != null) {
Review comment:
It doesn't seem to return a range of 1. If rangeLength is 0, S3 throws
an error about it being an invalid range and the file routes to the Failure
queue. A rangeLength of 1B downloads 1 byte as expected. Am I missing
something? I'll update the validator for rangeLength to use
createDataSizeBoundsValidator(1, Long.MAX_VALUE) so that we don't ever get to
the zero length error condition.
----------------------------------------------------------------
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:
[email protected]
