fireroundgithub commented on code in PR #382:
URL: https://github.com/apache/rocketmq-connect/pull/382#discussion_r1030098416


##########
connectors/rocketmq-connect-sftp/src/main/java/org/apache/rocketmq/connect/http/sink/SftpSourceTask.java:
##########
@@ -0,0 +1,120 @@
+package org.apache.rocketmq.connect.http.sink;
+
+import io.openmessaging.KeyValue;
+import io.openmessaging.connector.api.component.task.source.SourceTask;
+import io.openmessaging.connector.api.component.task.source.SourceTaskContext;
+import io.openmessaging.connector.api.data.ConnectRecord;
+import io.openmessaging.connector.api.data.RecordOffset;
+import io.openmessaging.connector.api.data.RecordPartition;
+import org.apache.commons.lang3.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
+import java.nio.file.FileSystemException;
+import java.util.*;
+
+import static org.apache.rocketmq.connect.http.sink.SftpConstant.*;
+
+public class SftpSourceTask extends SourceTask {
+
+    private final Logger log = 
LoggerFactory.getLogger(SftpConstant.LOGGER_NAME);
+
+    private SftpClient sftpClient;
+
+    private String filename;
+
+    private static final int MAX_NUMBER_SEND_CONNECT_RECORD_EACH_TIME = 2000;
+
+    @Override
+    public void init(SourceTaskContext sourceTaskContext) {
+        super.init(sourceTaskContext);
+    }
+
+    @Override
+    public void start(KeyValue config) {
+        String host = config.getString(SFTP_HOST_KEY);
+        int port = config.getInt(SFTP_PORT_KEY);
+        String username = config.getString(SFTP_USERNAME_KEY);
+        String password = config.getString(SFTP_PASSWORD_KEY);
+        String path = config.getString(SFTP_PATH_KEY);
+        filename = config.getString(SFTP_FILENAME_KEY);
+        sftpClient = new SftpClient(host, port, username, password, path);
+    }
+
+    @Override
+    public void stop() {
+    }

Review Comment:
   Are you think it will be better to close sftpClient when stopping a task?



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