nehsyc commented on pull request #15178: URL: https://github.com/apache/beam/pull/15178#issuecomment-881641458
> > Do you think we should keep the old behavior as an option in case some customers run into issues ? > > We should make it the default for runner v2. > > @reuvenlax Is it worth making this the default and requiring users to pass in a flag to maintain support for pipeline update for older versions? I assume that you meant making runner determined sharding default for runner v2. Older versions don't support runner determined sharding for unbounded data (strictly speaking, versions before 2.29.0) so fixed sharding option will be carried over on pipeline update. We could make new pipelines opt in by default. -- 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]
