phoerious commented on a change in pull request #15931:
URL: https://github.com/apache/beam/pull/15931#discussion_r768496977
##########
File path: sdks/python/apache_beam/io/aws/clients/s3/boto3_client.py
##########
@@ -92,27 +99,65 @@ def get_object_metadata(self, request):
boto_response['ContentLength'],
boto_response['ContentType'])
+ item.size = max(item.size - self._download_offset, 0)
return item
+ def get_stream(self, request, start):
Review comment:
Is there a guide somewhere how I can build and test the Python SDK? The
`./gradlew buildPython` job fails with totally non-descriptive virtualenv
errors on my machine. Hence I have to rely on your CI to debug stuff. That's
why there were two variable name inconsistencies, which I overlooked when I
copied the code from my own project to the Beam SDK code.
--
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]
