nifi-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #915: MINIFICPP-1376 Create PutS3Object processor
Date Wed, 21 Oct 2020 16:24:17 GMT

lordgamez commented on a change in pull request #915:
URL: https://github.com/apache/nifi-minifi-cpp/pull/915#discussion_r509426886



##########
File path: PROCESSORS.md
##########
@@ -1125,3 +1126,40 @@ In the list below, the names of required properties appear in bold.
Any other pr
 |success|All files are routed to success|
 
 
+## PutS3Object
+
+### Description
+
+Puts FlowFiles to an Amazon S3 Bucket. The upload uses either the PutS3Object method. The
PutS3Object method send the file in a single synchronous call, but it has a 5GB size limit.
Larger files sent using the multipart upload methods are currently not supported. The AWS
libraries select an endpoint URL based on the AWS region, but this can be overridden with
the 'Endpoint Override URL' property for use with other S3-compatible endpoints. The S3 API
specifies that the maximum file size for a PutS3Object upload is 5GB.
+### Properties
+
+In the list below, the names of required properties appear in bold. Any other properties
(not in bold) are considered optional. The table also indicates any default values, and whether
a property supports the NiFi Expression Language.
+
+| Name | Default Value | Allowable Values | Description |
+| - | - | - | - |
+|**Object Key**|${filename}||The key of the S3 object<br/>**Supports Expression Language:
true**|

Review comment:
       I searched for the `s3.permission` parameters in NiFi if any of the processors have
them as outputs, but I could not find any, only as the default input values of PutS3Object.
I am not sure, where these defaults come from, but I suppose in this case it would be more
advantageous to have support for agents without EL support if we cannot find any other references
to them. I removed the EL dependent default values in [bc6b752](https://github.com/apache/nifi-minifi-cpp/pull/915/commits/bc6b752e03ddd29dc150cbb598414cb332ca5592)




----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message