[GitHub] zentol commented on a change in pull request #7046: [FLINK-10803] Update the documentation to include changes to the S3 connector.

[GitBox <gi...@apache.org>]

zentol commented on a change in pull request #7046: [FLINK-10803] Update the documentation to include changes to the S3 connector.
URL: https://github.com/apache/flink/pull/7046#discussion_r231609764
 
 

 ##########
 File path: docs/dev/connectors/streamfile_sink.md
 ##########
 @@ -24,16 +24,24 @@ under the License.
 -->
 
 This connector provides a Sink that writes partitioned files to filesystems
-supported by the Flink `FileSystem` abstraction. Since in streaming the input
-is potentially infinite, the streaming file sink writes data into buckets. The
-bucketing behaviour is configurable but a useful default is time-based
+supported by the [Flink `FileSystem` abstraction]({{ site.baseurl}}/ops/filesystems.html).
+
+<span class="label label-danger">Important Note</span>: For S3, the `StreamingFileSink` 
+supports only Hadoop (not Presto). In case your job uses the `StreamingFileSink` to write to S3 but
 
 Review comment:
   does this page contain a link to presto somewhere? If not, please add one.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services