johnjcasey commented on code in PR #28853:
URL: https://github.com/apache/beam/pull/28853#discussion_r1415872379


##########
runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/SplittableParDo.java:
##########
@@ -669,6 +670,15 @@ public void output(RestrictionT part) {
                     public void outputWithTimestamp(RestrictionT part, Instant 
timestamp) {
                       throw new UnsupportedOperationException();
                     }
+
+                    @Override
+                    public void outputWindowedValue(

Review Comment:
   It might be easier to make this outputWindowedValue a default implementation 
for OutputReceiver to simplify this change for all the test implementations of 
OutputReceiver.



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