claudevdm commented on PR #38429: URL: https://github.com/apache/beam/pull/38429#issuecomment-4499422612
> > @tvalentyn @damccorm do you have throughts on this change? > > Maybe we should log a warning rather than throw error? There might be users who depend on the current behavior in some way. > > I think it is probably ok to disallow this. I can't really think of a case where it is valid, and it does cause lots of problems for beginners in my experience. > > Are there any valid use cases you can think of that do this? Can't really think about valid use cases, but it could break people who depend on this without really knowing. So calling out as a breaking change sounds good. -- 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]
