You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/05/02 22:24:51 UTC

[GitHub] [beam] Abacn commented on pull request #17380: [BEAM-14314][BEAM-9532] Add last_updated field in filesystem.FileMetaData

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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org