lukecwik commented on a change in pull request #16388:
URL: https://github.com/apache/beam/pull/16388#discussion_r776831641



##########
File path: 
examples/java/src/test/java/org/apache/beam/examples/cookbook/BigQueryTornadoesIT.java
##########
@@ -60,7 +60,7 @@
 
   @BeforeClass
   public static void setUp() {
-    PipelineOptionsFactory.register(BigQueryTornadoesITOptions.class);
+    PipelineOptionsFactory.register(TestPipelineOptions.class);

Review comment:
       Not a bug. Each PipelineOptions instance must be internally consistent 
(e.g. types and default values have to be the same). Registering a 
PipelineOptions makes it such that all PipelineOptions instances have to 
conform to this new option. It must be that the ITs used to run in different 
JVMs and now are running in the same JVM.




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