You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/07/15 22:37:59 UTC

[GitHub] [incubator-seatunnel] kezhenxu94 commented on a diff in pull request #2180: Delete a repeated dependency libary

kezhenxu94 commented on code in PR #2180:
URL: https://github.com/apache/incubator-seatunnel/pull/2180#discussion_r922559292


##########
pom.xml:
##########
@@ -864,11 +864,6 @@
                 <artifactId>curator-recipes</artifactId>
                 <version>${curator.version}</version>
             </dependency>
-            <dependency>

Review Comment:
   > https://github.com/apache/incubator-seatunnel/blob/dev/tools/dependencies/known-dependencies.txt#L110 we used the curator-recipes, why did we need to delete this? The fixed version is a very important thing when there are multiple versions.
   
   @CalvinKirs because there is still a same dependency above at line 862 🙈



-- 
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: commits-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org