gradle: remove deps.toml (airbytehq#45160)

Marius Posta · web-flow · commit dd2d44357200 · 2024-09-05T08:44:44.000-05:00
diff --git a/airbyte-cdk/java/airbyte-cdk/db-destinations/build.gradle b/airbyte-cdk/java/airbyte-cdk/db-destinations/build.gradle
@@ -10,8 +10,8 @@ dependencies {
 
     implementation 'io.aesy:datasize:1.0.0'
 
-    testFixturesCompileOnly libs.lombok
-    testFixturesAnnotationProcessor libs.lombok
+    testFixturesCompileOnly 'org.projectlombok:lombok:1.18.30'
+    testFixturesAnnotationProcessor 'org.projectlombok:lombok:1.18.30'
     testFixturesImplementation project(':airbyte-cdk:java:airbyte-cdk:airbyte-cdk-dependencies')
     testFixturesImplementation testFixtures(project(':airbyte-cdk:java:airbyte-cdk:airbyte-cdk-dependencies'))
     testFixturesImplementation project(':airbyte-cdk:java:airbyte-cdk:airbyte-cdk-core')
diff --git a/airbyte-ci/connectors/pipelines/README.md b/airbyte-ci/connectors/pipelines/README.md
@@ -843,6 +843,7 @@ airbyte-ci connectors --language=low-code migrate-to-manifest-only
 
 | Version | PR                                                         | Description                                                                                                                  |
 | ------- | ---------------------------------------------------------- |------------------------------------------------------------------------------------------------------------------------------|
+| 4.35.1  | [#45160](https://github.com/airbytehq/airbyte/pull/45160)  | Remove deps.toml dependency for java connectors.                                                                             |
 | 4.35.0  | [#44879](https://github.com/airbytehq/airbyte/pull/44879)  | Mount `components.py` when building manifest-only connector image                                                            |
 | 4.34.2  | [#44786](https://github.com/airbytehq/airbyte/pull/44786)  | Pre-emptively skip archived connectors when searching for modified files                                                     |
 | 4.34.1  | [#44557](https://github.com/airbytehq/airbyte/pull/44557)  | Conditionally propagate parameters in manifest-only migration                                                                |
diff --git a/airbyte-ci/connectors/pipelines/pipelines/airbyte_ci/steps/gradle.py b/airbyte-ci/connectors/pipelines/pipelines/airbyte_ci/steps/gradle.py
@@ -83,7 +83,6 @@ async def _run(self, *args: Any, **kwargs: Any) -> StepResult:
             ".root",
             ".env",
             "build.gradle",
-            "deps.toml",
             "gradle.properties",
             "gradle",
             "gradlew",
diff --git a/airbyte-ci/connectors/pipelines/pyproject.toml b/airbyte-ci/connectors/pipelines/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "pipelines"
-version = "4.35.0"
+version = "4.35.1"
 description = "Packaged maintained by the connector operations team to perform CI for connectors' pipelines"
 authors = ["Airbyte <contact@airbyte.io>"]
 
diff --git a/airbyte-integrations/connectors/destination-clickhouse-strict-encrypt/build.gradle b/airbyte-integrations/connectors/destination-clickhouse-strict-encrypt/build.gradle
@@ -29,8 +29,8 @@ dependencies {
     implementation 'com.clickhouse:clickhouse-jdbc:0.3.2-patch10:all'
 
     // https://mvnrepository.com/artifact/org.testcontainers/clickhouse
-    testImplementation libs.testcontainers.clickhouse
+    testImplementation 'org.testcontainers:clickhouse:1.19.0'
 
     // https://mvnrepository.com/artifact/org.testcontainers/clickhouse
-    integrationTestJavaImplementation libs.testcontainers.clickhouse
+    integrationTestJavaImplementation 'org.testcontainers:clickhouse:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-clickhouse/build.gradle b/airbyte-integrations/connectors/destination-clickhouse/build.gradle
@@ -28,8 +28,8 @@ dependencies {
     implementation 'com.clickhouse:clickhouse-jdbc:0.3.2-patch10:all'
 
     // https://mvnrepository.com/artifact/org.testcontainers/clickhouse
-    testImplementation libs.testcontainers.clickhouse
+    testImplementation 'org.testcontainers:clickhouse:1.19.0'
 
     // https://mvnrepository.com/artifact/org.testcontainers/clickhouse
-    integrationTestJavaImplementation libs.testcontainers.clickhouse
+    integrationTestJavaImplementation 'org.testcontainers:clickhouse:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-elasticsearch-strict-encrypt/build.gradle b/airbyte-integrations/connectors/destination-elasticsearch-strict-encrypt/build.gradle
@@ -40,6 +40,6 @@ dependencies {
 
     // MIT
     // https://www.testcontainers.org/
-    testImplementation libs.testcontainers.elasticsearch
-    integrationTestJavaImplementation libs.testcontainers.elasticsearch
+    testImplementation 'org.testcontainers:elasticsearch:1.19.0'
+    integrationTestJavaImplementation 'org.testcontainers:elasticsearch:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-elasticsearch/build.gradle b/airbyte-integrations/connectors/destination-elasticsearch/build.gradle
@@ -39,6 +39,6 @@ dependencies {
 
     // MIT
     // https://www.testcontainers.org/
-    testImplementation libs.testcontainers.elasticsearch
-    integrationTestJavaImplementation libs.testcontainers.elasticsearch
+    testImplementation 'org.testcontainers:elasticsearch:1.19.0'
+    integrationTestJavaImplementation 'org.testcontainers:elasticsearch:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-iceberg/build.gradle b/airbyte-integrations/connectors/destination-iceberg/build.gradle
@@ -55,8 +55,8 @@ dependencies {
         }
     }
 
-    testImplementation libs.testcontainers.postgresql
-    integrationTestJavaImplementation libs.testcontainers.postgresql
+    testImplementation 'org.testcontainers:postgresql:1.19.0'
+    integrationTestJavaImplementation 'org.testcontainers:postgresql:1.19.0'
 
     testImplementation 'org.mockito:mockito-inline:4.7.0'
 }
diff --git a/airbyte-integrations/connectors/destination-kafka/build.gradle b/airbyte-integrations/connectors/destination-kafka/build.gradle
@@ -28,5 +28,5 @@ dependencies {
     implementation 'org.apache.kafka:kafka-clients:2.8.0'
     implementation 'org.apache.kafka:connect-json:2.8.0'
 
-    integrationTestJavaImplementation libs.testcontainers.kafka
+    integrationTestJavaImplementation 'org.testcontainers:kafka:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-mssql-strict-encrypt/build.gradle b/airbyte-integrations/connectors/destination-mssql-strict-encrypt/build.gradle
@@ -34,6 +34,6 @@ dependencies {
     implementation 'com.microsoft.sqlserver:mssql-jdbc:8.4.1.jre14'
 
     testImplementation 'org.apache.commons:commons-lang3:3.11'
-    testImplementation libs.testcontainers.mssqlserver
+    testImplementation 'org.testcontainers:mssqlserver:1.19.0'
 
 }
diff --git a/airbyte-integrations/connectors/destination-mssql/build.gradle b/airbyte-integrations/connectors/destination-mssql/build.gradle
@@ -33,5 +33,5 @@ dependencies {
     implementation 'com.microsoft.sqlserver:mssql-jdbc:8.4.1.jre14'
 
     testImplementation 'org.apache.commons:commons-lang3:3.11'
-    testImplementation libs.testcontainers.mssqlserver
+    testImplementation 'org.testcontainers:mssqlserver:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-mysql-strict-encrypt/build.gradle b/airbyte-integrations/connectors/destination-mysql-strict-encrypt/build.gradle
@@ -29,5 +29,5 @@ dependencies {
     implementation 'mysql:mysql-connector-java:8.0.22'
 
     integrationTestJavaImplementation project(':airbyte-integrations:connectors:destination-mysql')
-    integrationTestJavaImplementation libs.testcontainers.mysql
+    integrationTestJavaImplementation 'org.testcontainers:mysql:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-mysql/build.gradle b/airbyte-integrations/connectors/destination-mysql/build.gradle
@@ -27,6 +27,6 @@ application {
 
 dependencies {
     implementation 'mysql:mysql-connector-java:8.0.22'
-    integrationTestJavaImplementation libs.testcontainers.mysql
-    testFixturesApi libs.testcontainers.mysql
+    integrationTestJavaImplementation 'org.testcontainers:mysql:1.19.0'
+    testFixturesApi 'org.testcontainers:mysql:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-oracle-strict-encrypt/build.gradle b/airbyte-integrations/connectors/destination-oracle-strict-encrypt/build.gradle
@@ -33,7 +33,7 @@ dependencies {
     implementation "com.oracle.database.jdbc:ojdbc8-production:19.7.0.0"
 
     testImplementation 'org.apache.commons:commons-lang3:3.11'
-    testImplementation libs.testcontainers.oracle.xe
+    testImplementation 'org.testcontainers:oracle-xe:1.19.0'
 
     integrationTestJavaImplementation project(':airbyte-integrations:connectors:destination-oracle')
 }
diff --git a/airbyte-integrations/connectors/destination-oracle/build.gradle b/airbyte-integrations/connectors/destination-oracle/build.gradle
@@ -32,5 +32,5 @@ dependencies {
     implementation "com.oracle.database.jdbc:ojdbc8-production:19.7.0.0"
 
     testImplementation 'org.apache.commons:commons-lang3:3.11'
-    testImplementation libs.testcontainers.oracle.xe
+    testImplementation 'org.testcontainers:oracle-xe:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-redis/build.gradle b/airbyte-integrations/connectors/destination-redis/build.gradle
@@ -37,5 +37,5 @@ dependencies {
     // https://mvnrepository.com/artifact/org.assertj/assertj-core
     testImplementation "org.assertj:assertj-core:${assertVersion}"
     // https://mvnrepository.com/artifact/org.testcontainers/testcontainers
-    testImplementation libs.testcontainers
+    testImplementation 'org.testcontainers:testcontainers:1.19.0'
 }
diff --git a/airbyte-integrations/connectors/destination-s3-glue/build.gradle b/airbyte-integrations/connectors/destination-s3-glue/build.gradle
@@ -27,6 +27,6 @@ dependencies {
     // https://mvnrepository.com/artifact/com.amazonaws/aws-java-sdk-glue
     implementation 'com.amazonaws:aws-java-sdk-glue:1.12.334'
 
-    implementation libs.aws.java.sdk.s3
-    implementation libs.s3
+    implementation 'com.amazonaws:aws-java-sdk-s3:1.12.6'
+    implementation 'software.amazon.awssdk:s3:2.20.20'
 }
diff --git a/deps.toml b/deps.toml
diff --git a/settings.gradle b/settings.gradle
@@ -102,12 +102,6 @@ dependencyResolutionManagement {
             }
         }
     }
-
-    versionCatalogs {
-        libs {
-            from(files("deps.toml"))
-        }
-    }
 }
 
 gradleEnterprise {

Original file line number	Diff line number	Diff line change
`@@ -29,8 +29,8 @@ dependencies {`
`29`	`29`	`implementation 'com.clickhouse:clickhouse-jdbc:0.3.2-patch10:all'`
`30`	`30`
`31`	`31`	`// https://mvnrepository.com/artifact/org.testcontainers/clickhouse`
`32`		`- testImplementation libs.testcontainers.clickhouse`
	`32`	`+ testImplementation 'org.testcontainers:clickhouse:1.19.0'`
`33`	`33`
`34`	`34`	`// https://mvnrepository.com/artifact/org.testcontainers/clickhouse`
`35`		`- integrationTestJavaImplementation libs.testcontainers.clickhouse`
	`35`	`+ integrationTestJavaImplementation 'org.testcontainers:clickhouse:1.19.0'`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,8 @@ dependencies {`
`28`	`28`	`implementation 'com.clickhouse:clickhouse-jdbc:0.3.2-patch10:all'`
`29`	`29`
`30`	`30`	`// https://mvnrepository.com/artifact/org.testcontainers/clickhouse`
`31`		`- testImplementation libs.testcontainers.clickhouse`
	`31`	`+ testImplementation 'org.testcontainers:clickhouse:1.19.0'`
`32`	`32`
`33`	`33`	`// https://mvnrepository.com/artifact/org.testcontainers/clickhouse`
`34`		`- integrationTestJavaImplementation libs.testcontainers.clickhouse`
	`34`	`+ integrationTestJavaImplementation 'org.testcontainers:clickhouse:1.19.0'`
`35`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,6 @@ dependencies {`
`40`	`40`
`41`	`41`	`// MIT`
`42`	`42`	`// https://www.testcontainers.org/`
`43`		`- testImplementation libs.testcontainers.elasticsearch`
`44`		`- integrationTestJavaImplementation libs.testcontainers.elasticsearch`
	`43`	`+ testImplementation 'org.testcontainers:elasticsearch:1.19.0'`
	`44`	`+ integrationTestJavaImplementation 'org.testcontainers:elasticsearch:1.19.0'`
`45`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,6 @@ dependencies {`
`39`	`39`
`40`	`40`	`// MIT`
`41`	`41`	`// https://www.testcontainers.org/`
`42`		`- testImplementation libs.testcontainers.elasticsearch`
`43`		`- integrationTestJavaImplementation libs.testcontainers.elasticsearch`
	`42`	`+ testImplementation 'org.testcontainers:elasticsearch:1.19.0'`
	`43`	`+ integrationTestJavaImplementation 'org.testcontainers:elasticsearch:1.19.0'`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,8 @@ dependencies {`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`
`58`		`- testImplementation libs.testcontainers.postgresql`
`59`		`- integrationTestJavaImplementation libs.testcontainers.postgresql`
	`58`	`+ testImplementation 'org.testcontainers:postgresql:1.19.0'`
	`59`	`+ integrationTestJavaImplementation 'org.testcontainers:postgresql:1.19.0'`
`60`	`60`
`61`	`61`	`testImplementation 'org.mockito:mockito-inline:4.7.0'`
`62`	`62`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,5 +28,5 @@ dependencies {`
`28`	`28`	`implementation 'org.apache.kafka:kafka-clients:2.8.0'`
`29`	`29`	`implementation 'org.apache.kafka:connect-json:2.8.0'`
`30`	`30`
`31`		`- integrationTestJavaImplementation libs.testcontainers.kafka`
	`31`	`+ integrationTestJavaImplementation 'org.testcontainers:kafka:1.19.0'`
`32`	`32`	`}`