[GitHub] [beam] iemejia commented on pull request #11715: [BEAM-9977] Implement GrowableOffsetRangeTracker

2020-05-21 Thread GitBox
iemejia commented on pull request #11715: URL: https://github.com/apache/beam/pull/11715#issuecomment-632434300 Thanks for answering and for the clear explaination @boyuanzz I would have tended towards having the additional complexity in `OffsetRangeTracker` just because it is the

[GitHub] [beam] iemejia commented on pull request #11715: [BEAM-9977] Implement GrowableOffsetRangeTracker

2020-05-21 Thread GitBox
iemejia commented on pull request #11715: URL: https://github.com/apache/beam/pull/11715#issuecomment-632402839 Now that this is merged. Can I ask a question. What is the intrinsic limitation that did not allow old `OffsetRangeTracker` to be refactored for this use case? or why we want to