HeartSaVioR commented on a change in pull request #30714:
URL: https://github.com/apache/spark/pull/30714#discussion_r575867762
##########
File path: docs/cloud-integration.md
##########
@@ -49,7 +49,6 @@ They cannot be used as a direct replacement for a cluster
filesystem such as HDF
Key differences are:
-* Changes to stored objects may not be immediately visible, both in directory
listings and actual data access.
Review comment:
The line deletion is based on the fact S3 is now strong consistency, but
that also means we only consider these three vendors. (Probably you're
considering more like S3 compatible implementations, but you've also mentioned
you don't consider the chinese cloud stores so it can't be exhaustive.) Why not
explicitly saying it and update the description tied to these vendors? We would
never be able to consider all possible implementations and for some minority
the description may be wrong. Let's just make it clear.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
