iemejia commented on pull request #11891:
URL: https://github.com/apache/beam/pull/11891#issuecomment-638017998


   I expected that vendored-grpc could be coming via some transitive dependency 
but it wasn't
   
https://repo1.maven.org/maven2/org/apache/beam/beam-sdks-java-core/2.21.0/beam-sdks-java-core-2.21.0.pom
   
   Maybe we should revert this, the only positive outcome I can see of this now 
is if at some moment `sdks/java/core` wants to act explictly on gRPC services 
which I doubt save for the `External` transform but this transform can easily 
be an extension to avoid adding the 20M in core.


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