lhotari commented on a change in pull request #9729:
URL: https://github.com/apache/pulsar/pull/9729#discussion_r587660459



##########
File path: pom.xml
##########
@@ -124,7 +124,7 @@ flexible messaging model and an intuitive client 
API.</description>
     <typetools.version>0.5.0</typetools.version>
     <protobuf3.version>3.11.4</protobuf3.version>
     <protoc3.version>${protobuf3.version}</protoc3.version>
-    <grpc.version>1.18.0</grpc.version>
+    <grpc.version>1.33.0</grpc.version>

Review comment:
       It would be useful to switch to use gprc-bom also in Pulsar, in the 
similar way as it was done for Bookkeeper: 
https://github.com/apache/bookkeeper/blob/7327e3b1c636ed42bdbe231fc30bcb73e527fee8/pom.xml#L462-L468
 . 
   gprc-bom became available in 1.25 and that's why it hasn't been used before.




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