litiliu commented on code in PR #9683:
URL: https://github.com/apache/seatunnel/pull/9683#discussion_r2297521474


##########
seatunnel-shade/seatunnel-hadoop3-3.1.4-uber/pom.xml:
##########
@@ -46,6 +46,13 @@
             <groupId>org.apache.hadoop</groupId>
             <artifactId>hadoop-client</artifactId>
             <version>${hadoop3.version}</version>
+            <!--exclude the okHttp 2.7.5-->
+            <exclusions>
+                <exclusion>
+                    <groupId>com.squareup.okhttp</groupId>
+                    <artifactId>okhttp</artifactId>
+                </exclusion>
+            </exclusions>

Review Comment:
   the support of okhttp 4.9.* only start from hadoop 
[3.4.0](https://issues.apache.org/jira/issues/?jql=project+%3D+HDFS+AND+fixVersion+%3D+3.4.0),
 
[3.3.4](https://issues.apache.org/jira/issues/?jql=project+%3D+HDFS+AND+fixVersion+%3D+3.3.4)
   , the previous hadoop versions are using a pretty old version okhttp 2.7.5.  
 see [HDFS-16453](https://issues.apache.org/jira/browse/HDFS-16453)
   
   Would it be better to updating the hadoop version(3.1.4->3.4.1) and the 
okhttp(2.7.5->4.12) at the same PR?
   
   



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