rmetzger commented on a change in pull request #12846:
URL: https://github.com/apache/flink/pull/12846#discussion_r458593290



##########
File path: flink-connectors/flink-connector-gcp-pubsub/pom.xml
##########
@@ -56,18 +56,22 @@ under the License.
                        <scope>provided</scope>
                </dependency>
 
+               <dependency>
+                       <groupId>com.google.cloud</groupId>
+                       <artifactId>google-cloud-core</artifactId>
+                       <!-- Version is pulled from google-cloud-bom (loaded 
via the libraries-bom) -->
+               </dependency>
+
                <dependency>
                        <groupId>com.google.cloud</groupId>
                        <artifactId>google-cloud-pubsub</artifactId>
-                       <!-- Version is pulled from google-cloud-bom -->
-                       <exclusions>
-                               <!-- Exclude an old version of guava that is 
being pulled
-                in by a transitive dependency of google-api-client -->
-                               <exclusion>
-                                       <groupId>com.google.guava</groupId>
-                                       <artifactId>guava-jdk5</artifactId>
-                               </exclusion>
-                       </exclusions>
+                       <!-- Version is pulled from google-cloud-bom (loaded 
via the libraries-bom) -->
+               </dependency>
+
+               <dependency>
+                       <groupId>com.google.api.grpc</groupId>
+                       <artifactId>grpc-google-cloud-pubsub-v1</artifactId>
+                       <!-- Version is pulled from google-cloud-bom (loaded 
via the libraries-bom) -->

Review comment:
       Mh, ok :( 




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