Abacn commented on PR #17380:
URL: https://github.com/apache/beam/pull/17380#issuecomment-1115429991

   > this mostly LGTM - though I would like to make sure we verify that the API 
clients return what they're supposed to return
   
   Thanks for feedboacks! For hadoop filesystem there is an existing 
integration test in which I checked the desired last_updated field was returned 
in correct unit and value. For others (aws, azure, hadoop, gcs) confirmed with 
API reference and all filesystem implementations are tested in unit-test level.


-- 
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]

Reply via email to