ihji commented on a change in pull request #13729:
URL: https://github.com/apache/beam/pull/13729#discussion_r591883441
##########
File path: sdks/python/apache_beam/runners/portability/expansion_service_test.py
##########
@@ -298,6 +324,10 @@ def main(unused_argv):
PipelineOptions(
["--experiments", "beam_fn_api", "--sdk_location",
"container"])),
server)
+ beam_artifact_api_pb2_grpc.add_ArtifactRetrievalServiceServicer_to_server(
Review comment:
I tried in a way that you described but had a problem that popen (for
fetching dependencies) didn't work well in grpc callbacks. We could try it
again in the future.
----------------------------------------------------------------
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]
