riteshghorse commented on a change in pull request #15430:
URL: https://github.com/apache/beam/pull/15430#discussion_r700384969



##########
File path: sdks/go/pkg/beam/core/graph/window/trigger.go
##########
@@ -35,3 +35,38 @@ const (
        NeverTrigger                           string = "Trigger_Never_"
        AfterSynchronizedProcessingTimeTrigger string = 
"Trigger_AfterSynchronizedProcessingTime_"
 )
+
+// TriggerDefault constructs a default trigger that fires after the end of 
window.
+// No provision for late arriving data.

Review comment:
       Yes it fires once. I haven't tested this for late data as I wasn't able 
to advance processing time on flink runner. 




-- 
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: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to