[ 
https://issues.apache.org/jira/browse/BEAM-989?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15716143#comment-15716143
 ] 

Daniel Halperin commented on BEAM-989:
--------------------------------------

There was a pretty long discussion on 
https://github.com/apache/incubator-beam/pull/826 (and Luke and I did some pair 
code review to try to figure this out) and we could not come up with any decent 
ways to test anything involving {{BatchRequest}} because of the way the test 
classes are structured.

Do you have specific improvements to make here?

Otherwise, suggest closing as infeasible.

> GcsUtil batch remove() and copy() need tests.
> ---------------------------------------------
>
>                 Key: BEAM-989
>                 URL: https://issues.apache.org/jira/browse/BEAM-989
>             Project: Beam
>          Issue Type: Test
>          Components: sdk-java-gcp
>            Reporter: Pei He
>            Assignee: Pei He
>
> remove(), copy(), and their callbacks are not tested with executeBatches().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to