damccorm commented on PR #35415: URL: https://github.com/apache/beam/pull/35415#issuecomment-3045865591
> > I think this is a Dataflow issue and should be addressed at that level. It looks like we're not correctly setting this on pipeline update in all cases. > > Will my PR mitigate this issue or it does not matter since it will be override by Dataflow? It will probably mitigate this issue, but (a) there is probably a general bug which is being introduced by Dataflow here, and (b) this feature is meant to provide safety guarantees when we update which we are losing. -- 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
