lukecwik commented on pull request #11972: URL: https://github.com/apache/beam/pull/11972#issuecomment-642233639
> > I'm curious with which runner we will run into this issue. > > All portable runners as far as I know (confirmed for Flink, Spark, Dataflow Python streaming). > > > The problem is that protos default isn't null but the default empty instance so the check should really be whether it is not the default instance. > > Thanks Luke, looks like `message.HasField` is what we want here. Can you also fix the state one as well (beam/sdks/python/apache_beam/runners/worker/sdk_worker.py Line 691)? ---------------------------------------------------------------- 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]
