You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2020/07/15 02:10:50 UTC

[GitHub] [beam] lukecwik commented on pull request #12262: [BEAM-10491] Simplify PeriodicSequence generator to use OffsetRanges with whole whole numbers

lukecwik commented on pull request #12262:
URL: https://github.com/apache/beam/pull/12262#issuecomment-658503845


   R: @Ardagan @pabloem 
   
   I wanted to simplify the existing implementation because users will look at existing code to see how to use it. Note that the tests won't pass with the corresponding fixes to OffsetRestrictionTracker in #12260


----------------------------------------------------------------
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:
users@infra.apache.org