brucearctor edited a comment on pull request #16578: URL: https://github.com/apache/beam/pull/16578#issuecomment-1048290443
Thanks, Cham. Even if hard, doesn't mean not worth doing. **My general philosophy is that if it is important to test once before merging then should be part of ongoing tests....**, so will at least try that. Though, it doesn't seem to make sense to test the STORAGE_API_AT_LEAST_ONCE , since the test might fail ( if configured to expect exactly-once ), even though the API would be working as 'expected'/advertised. -- 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]
