z3d1k commented on code in PR #20476:
URL: https://github.com/apache/flink/pull/20476#discussion_r940206925


##########
flink-end-to-end-tests/flink-glue-schema-registry-json-test/pom.xml:
##########
@@ -32,56 +32,83 @@ under the License.
        <packaging>jar</packaging>
 
        <properties>
-               <guava.version>29.0-jre</guava.version>
-               <netty.version>4.1.68.Final</netty.version>
+               <aws.sdk.version>1.12.276</aws.sdk.version>
+               <aws.sdkv2.version>2.17.247</aws.sdkv2.version>
                <kotlin.version>1.3.50</kotlin.version>
-               <classgraph.version>4.8.120</classgraph.version>
        </properties>
 
-       <!-- ============================= -->
-       <!-- DEPENDENCY MANAGEMENT -->
-       <!-- ============================= -->
        <dependencyManagement>
                <dependencies>
-                       <!-- dependencies to solve enforcer check issue -->
-                       <!-- can be removed once new version of Glue Schema 
Registry releases -->
-
                        <dependency>
-                               <groupId>com.google.guava</groupId>
-                               <artifactId>guava</artifactId>
-                               <version>${guava.version}</version>
+                               <groupId>software.amazon.awssdk</groupId>
+                               <artifactId>bom</artifactId>
+                               <version>${aws.sdkv2.version}</version>
+                               <type>pom</type>
+                               <scope>import</scope>
                        </dependency>
-
                        <dependency>
-                               <groupId>io.netty</groupId>
-                               <artifactId>netty-handler</artifactId>
-                               <version>${netty.version}</version>
+                               <groupId>com.amazonaws</groupId>
+                               <artifactId>aws-java-sdk-bom</artifactId>
+                               <version>${aws.sdk.version}</version>
+                               <type>pom</type>
+                               <scope>import</scope>
                        </dependency>
 
+                       <!-- For dependency convergence -->
                        <dependency>
-                               <groupId>io.netty</groupId>
-                               <artifactId>netty-codec-http</artifactId>
-                               <version>${netty.version}</version>
+                               <groupId>com.google.protobuf</groupId>
+                               <artifactId>protobuf-java</artifactId>
+                               <version>3.19.4</version>
+                       </dependency>
+                       <dependency>
+                               <groupId>com.google.guava</groupId>
+                               <artifactId>guava</artifactId>
+                               <version>30.0-jre</version>
+                       </dependency>
+                       <dependency>
+                               <groupId>org.jetbrains.kotlin</groupId>
+                               <artifactId>kotlin-reflect</artifactId>
+                               <version>${kotlin.version}</version>
                        </dependency>
-
                        <dependency>
                                <groupId>org.jetbrains.kotlin</groupId>
                                <artifactId>kotlin-stdlib</artifactId>
                                <version>${kotlin.version}</version>
                        </dependency>
-
                        <dependency>
                                <groupId>org.jetbrains.kotlin</groupId>
                                <artifactId>kotlin-stdlib-common</artifactId>
                                <version>${kotlin.version}</version>
                        </dependency>
-
+                       <dependency>
+                               <groupId>org.jetbrains.kotlin</groupId>
+                               <artifactId>kotlin-stdlib-jdk8</artifactId>
+                               <version>${kotlin.version}</version>
+                       </dependency>
+                       <dependency>
+                               <groupId>org.jetbrains.kotlinx</groupId>
+                               
<artifactId>kotlinx-serialization-core-jvm</artifactId>
+                               <version>1.0.1</version>
+                       </dependency>
+                       <dependency>
+                               <groupId>org.checkerframework</groupId>
+                               <artifactId>checker-qual</artifactId>
+                               <version>3.5.0</version>
+                       </dependency>
                        <dependency>
                                <groupId>io.github.classgraph</groupId>
                                <artifactId>classgraph</artifactId>
-                               <version>${classgraph.version}</version>
+                               <version>4.8.120</version>
+                       </dependency><dependency>
+                       <groupId>com.squareup.okio</groupId>
+                       <artifactId>okio</artifactId>
+                       <version>2.8.0</version>
+               </dependency>

Review Comment:
   Block indentation is off



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to