-
Notifications
You must be signed in to change notification settings - Fork 31
Add integration test for IdentityByState #174
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
Fixes #70 |
I signed it. On Tue, Mar 8, 2016 at 10:36 PM, googlebot notifications@github.com wrote:
Regards, |
} | ||
|
||
@Test | ||
public void testStreamingLocal() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
useGrpc
isn't actually an argument to this pipeline, are you able to successfully run this test?
I get error java.lang.IllegalArgumentException: Class interface com.google.cloud.genomics.dataflow.pipelines.IdentityByState$Options missing a property named 'useGrpc'.
when I run it.
Also I found I needed to do the following when I ran into this issue:
(1) grab a more recent version of alpn and stick it in ./lib
(2) update the activation section in pom.xml to use the newer version
@nitag this is definitely on the right track, thank you! |
pulling changes from googlegenomics
pulling from master into ibs-it branch
Thanks for the comments. Submitting a new pull request with the merged changes. |
Integration test for IdentityByState (issue 70)