deps: update grpc to 1.30.0, protobuf to 3.12.2

[chingor13]

aftects: googleapis/java-pubsub#217

[codecov]

Codecov Report

Merging #1125 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1125   +/-   ##
=========================================
  Coverage     78.66%   78.66%           
  Complexity     1169     1169           
=========================================
  Files           204      204           
  Lines          5189     5189           
  Branches        417      417           
=========================================
  Hits           4082     4082           
  Misses          933      933           
  Partials        174      174           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecbab4a...daaa4a7. Read the comment docs.

[elharo]

I think protobuf 3.12.2 is out

[chingor13]

I was trying to match what's in libraries-bom. Perhaps we should bump to 1.30.0 and 3.12.2 respectively both here and in libraries-bom?

[suztomo]

Class com.ning.compress.lzf.LZFChunk is not found;
  referenced by 1 class file
    io.grpc.netty.shaded.io.netty.handler.codec.compression.LzfEncoder (io.grpc:grpc-netty-shaded:1.30.0)

io.grpc:grpc-netty-shaded:1.30.0 is at:
  com.google.api:gax-grpc:jar:1.56.0 / io.grpc:grpc-netty-shaded:1.30.0 (compile)
  and 1 dependency path.


> Task :gax-grpc:checkJavaLinkage FAILED

Checking whether this is in the exclusion file.

[suztomo]

The target & source pair is not listed. It's a new linkage error in the dependency.

Would you add the following element to https://github.com/googleapis/gax-java/blob/chingor13-patch-1/gax-grpc/linkage-checker-exclusion.xml#L171 ?

  <LinkageError>
    <Target>
      <Class name="com.ning.compress.lzf.LZFChunk"/>
    </Target>
    <Source>
      <Class name="io.grpc.netty.shaded.io.netty.handler.codec.compression.LzfEncoder"/>
    </Source>
  </LinkageError>

[elharo]

Can we fix this instead of excluding it? This is a true positive. Do we care?

[suztomo]

@elharo I don't think we can fix it. It is true positive while gRPC users do not suffer from this. The reference is from shaded Netty for the (unused) compression algorithm.

[suztomo]

9 successful checks

Thank you.

[reachlakstar]

I am using Apache beam with grpc and dataflow. 1.29.0 and 3.12.2, facing same issue. any solution for this.
java.lang.ClassNotFoundException: io.grpc.internal.BaseDnsNameResolverProvider

[suztomo]

@reachlakstar The class does not seem gax-java. Would you create an issue in Beam's issue management with your pom.xml or build.gradle, and give me the URL?