kennknowles commented on pull request #11587:
URL: https://github.com/apache/beam/pull/11587#issuecomment-624745905


   Is there a "normal" pattern that should be used for cases like this? Having 
an import be sometimes `None` seems very odd and potentially confusing to 
someone coming along later. Maybe a Jira to track a further fix? (or if "this 
is fine" then OK)


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to