reuvenlax commented on a change in pull request #11739:
URL: https://github.com/apache/beam/pull/11739#discussion_r432878308



##########
File path: 
sdks/java/core/src/test/java/org/apache/beam/sdk/transforms/ParDoTest.java
##########
@@ -3539,6 +3539,7 @@ public void onTimer() {}
     @Category({
       ValidatesRunner.class,
       UsesTimersInParDo.class,
+      UsesUnboundedPCollections.class,

Review comment:
       I think it was so that it would run on a streaming runner (especially 
the Dataflow runner). ValidatesRunner tests that use TestStream don't run on 
Dataflow today I believe, and we have several other places in our test codebase 
where this pattern is used.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to