Abacn commented on code in PR #17380:
URL: https://github.com/apache/beam/pull/17380#discussion_r863182185
##########
sdks/python/apache_beam/io/aws/s3io.py:
##########
@@ -464,6 +459,49 @@ def exists(self, path):
# We re-raise all other exceptions
raise
+ def _status(self, path):
Review Comment:
This function is also used by FileSystem.metadata implementations, in order
to obtain both file size and last updated fields at once (one request in web
filesystems). If the file size and last_updated is collected separately it
would double the api request for filesystem._list (used by match) and
regression may be introduced.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
