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

Reply via email to