source-sftp: adopt CDK 0.20.4 (airbytehq#35221)

ollionorg · Feb 26, 2024 · 08cfd2f · 08cfd2f
1 parent d7ef9b9
commit 08cfd2f
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 13 deletions.
diff --git a/airbyte-integrations/connectors/source-sftp/build.gradle b/airbyte-integrations/connectors/source-sftp/build.gradle
@@ -1,23 +1,13 @@
 plugins {
-    id 'application'
     id 'airbyte-java-connector'
 }
 
 airbyteJavaConnector {
-    cdkVersionRequired = '0.13.2'
+    cdkVersionRequired = '0.20.4'
     features = ['db-sources']
     useLocalCdk = false
 }
 
-//remove once upgrading the CDK version to 0.4.x or later
-java {
-    compileJava {
-        options.compilerArgs.remove("-Werror")
-    }
-}
-
-airbyteJavaConnector.addCdkDependencies()
-
 application {
     mainClass = 'io.airbyte.integrations.source.sftp.SftpSource'
 }
@@ -26,5 +16,5 @@ dependencies {
     implementation 'com.fasterxml.jackson.dataformat:jackson-dataformat-csv:2.13.2'
     implementation 'com.jcraft:jsch:0.1.55'
 
-    testImplementation libs.testcontainers
+    testImplementation 'org.testcontainers:testcontainers:1.19.4'
 }
diff --git a/airbyte-integrations/connectors/source-sftp/metadata.yaml b/airbyte-integrations/connectors/source-sftp/metadata.yaml
@@ -5,7 +5,7 @@ data:
   connectorSubtype: file
   connectorType: source
   definitionId: a827c52e-791c-4135-a245-e233c5255199
-  dockerImageTag: 0.2.1
+  dockerImageTag: 0.2.2
   dockerRepository: airbyte/source-sftp
   documentationUrl: https://docs.airbyte.com/integrations/sources/sftp
   githubIssueLabel: source-sftp

diff --git a/.../connectors/source-sftp/src/main/java/io/airbyte/integrations/source/sftp/SftpClient.java b/.../connectors/source-sftp/src/main/java/io/airbyte/integrations/source/sftp/SftpClient.java
@@ -90,6 +90,7 @@ public boolean isConnected() {
     return channelSftp != null && channelSftp.isConnected();
   }
 
+  @SuppressWarnings("rawtypes")
   public Vector lsFile(SupportedFileExtension fileExtension) {
     try {
       return channelSftp.ls("*." + fileExtension.typeName);

diff --git a/...connectors/source-sftp/src/main/java/io/airbyte/integrations/source/sftp/SftpCommand.java b/...connectors/source-sftp/src/main/java/io/airbyte/integrations/source/sftp/SftpCommand.java
@@ -79,6 +79,7 @@ public Map<String, JsonNode> getFilesSchemas() {
     return fileSchemas;
   }
 
+  @SuppressWarnings("unchecked")
   private Set<String> getFileNames() {
     checkIfConnected();
     Vector<LsEntry> entries = new Vector<>();

diff --git a/docs/integrations/sources/sftp.md b/docs/integrations/sources/sftp.md
@@ -108,6 +108,7 @@ More formats \(e.g. Apache Avro\) will be supported in the future.
 
 | Version | Date       | Pull Request                                             | Subject                                                |
 |:--------|:-----------|:---------------------------------------------------------|:-------------------------------------------------------|
+| 0.2.2 | 2024-02-13 | [35221](https://github.com/airbytehq/airbyte/pull/35221) | Adopt CDK 0.20.4 |
 | 0.2.1   | 2024-01-24 | [34453](https://github.com/airbytehq/airbyte/pull/34453) | bump CDK version                                       |
 | 0.2.0   | 2024-01-15 | [34265](https://github.com/airbytehq/airbyte/pull/34265) | Remove LEGACY state flag                               |
 | 0.1.2   | 2022-06-17 | [13864](https://github.com/airbytehq/airbyte/pull/13864) | Updated stacktrace format for any trace message errors |