Abacn commented on code in PR #35569:
URL: https://github.com/apache/beam/pull/35569#discussion_r2211471196


##########
sdks/python/apache_beam/transforms/periodicsequence.py:
##########
@@ -216,6 +217,21 @@ def expand(self, pcoll):
         | 'MapToTimestamped' >> beam.Map(lambda tt: TimestampedValue(tt, tt)))
 
 
+class RebaseMode(enum.Enum):

Review Comment:
   Java/Python PeriodicImpulse API now gradually deviate. In Java side there is 
a "stopAfter"option that make start timestamp gets stamped at the beginning of 
pipeline execution 



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