liferoad commented on PR #35521: URL: https://github.com/apache/beam/pull/35521#issuecomment-3046145342
> > R: @liferoad since this came out of #28061 > > I'm somewhat ambivalent about this one. I've seen issues where this warning has or would have helped. I've also seen people get confused by the wording. > > Instead of removing the warning, could we clarify the text so that it is obvious that you can ignore it if it is intentional? > > It'd be nice if there was an easy way to discriminate between whether this case might be an accident or if it's actually intended. As it stands, I'm not sure there's an obvious way to silence this warning as a DoFn author who really doesn't want to yield anything (I suppose they can just yield and empty generator but I wouldn't call that obvious) I would argue the good practice is to return something. -- 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