embulk-output-sftp 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c59819e8adf5918b012907c1649584986ca9f77b
-  data.tar.gz: b942eb8bcaf5a55f18feae4ae68a6c0d4ef96b92
+  metadata.gz: f2b11bf7261d43608476053f62b5425f45643c1a
+  data.tar.gz: dc84dd6e807ae8d0b6736eaa485ced869e8c83c2
 SHA512:
-  metadata.gz: 0abe7dba22a634303bc872806d348c2f2f380257c4ebf7cdd5de3e10bf56644e37e82e5ce72e49f8ac2c76169a77781c5aefdf96112f8bb28da37db8178cf97e
-  data.tar.gz: 4e4073715b98d80bf66fd1c24f846079a4faf73993ec7e619355309c2728bac6459183d053c45fcbb80d2cfbccc527ae466324f264d8260b1a52704e227603af
+  metadata.gz: 0025aef591b2cdb982df7d09a5b2225f1855b2e7ab97966d34cd61b74eaa1e1dcab848af0881a5f4a2159d5d6d9d39f99fd064a0ed9cf3cc686a4e9359b398e0
+  data.tar.gz: 5bb8ab885d48fb2e3a4af7a5db9e850d7c185e0d2d4de8897abaf5c2580fba1ec4e16c3a5c60be91239cb4db280420f1dcdb8f863bc0132e0ac0ba0a8f62ce20

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+0.1.4 (2017-12-21)
+- Fix: Disable remote temporary file rename logic
+  - https://github.com/embulk/embulk-output-sftp/pull/41
+- Enhance: Upgrade "commons-vfs2", "com.jcraft:jsch" and "commons-io:commons-io"
+  - https://github.com/embulk/embulk-output-sftp/pull/42
+
 0.1.3 (2017-11-07)
 - Enhance: Create temporary file and rename it after upload completed
   - https://github.com/embulk/embulk-output-sftp/pull/37

data/build.gradle

@@ -17,7 +17,7 @@ configurations {
 }
 
 group = "org.embulk.output.sftp"
-version = "0.1.3"
+version = "0.1.4"
 sourceCompatibility = 1.7
 targetCompatibility = 1.7
 
@@ -25,9 +25,9 @@ dependencies {
     compile  "org.embulk:embulk-core:0.8.6"
     provided "org.embulk:embulk-core:0.8.6"
     // compile "YOUR_JAR_DEPENDENCY_GROUP:YOUR_JAR_DEPENDENCY_MODULE:YOUR_JAR_DEPENDENCY_VERSION"
-    compile "org.apache.commons:commons-vfs2:2.1.1660580.2"
-    compile "commons-io:commons-io:2.5"
-    compile "com.jcraft:jsch:0.1.53"
+    compile "org.apache.commons:commons-vfs2:2.2"
+    compile "commons-io:commons-io:2.6"
+    compile "com.jcraft:jsch:0.1.54"
     testCompile "junit:junit:4.+"
     testCompile "org.embulk:embulk-core:0.8.6:tests"
     testCompile "org.embulk:embulk-standards:0.8.6"

data/src/main/java/org/embulk/output/sftp/SftpFileOutput.java

@@ -84,7 +84,12 @@ public class SftpFileOutput
         closeCurrentFile();
         String fileName = getOutputFilePath();
         String temporaryFileName = fileName + temporaryFileSuffix;
-        sftpUtils.uploadFile(tempFile, temporaryFileName);
+        /*
+          #37 causes permission failure while renaming remote file.
+          https://github.com/embulk/embulk-output-sftp/issues/40
+         */
+        //sftpUtils.uploadFile(tempFile, temporaryFileName);
+        sftpUtils.uploadFile(tempFile, fileName);
 
         Map<String, String> executedFiles = new HashMap<>();
         executedFiles.put("temporary_filename", temporaryFileName);

data/src/main/java/org/embulk/output/sftp/SftpFileOutputPlugin.java

@@ -14,7 +14,6 @@ import org.embulk.spi.TransactionalFileOutput;
 import org.embulk.spi.unit.LocalFile;
 
 import java.util.List;
-import java.util.Map;
 
 public class SftpFileOutputPlugin
         implements FileOutputPlugin
@@ -99,17 +98,21 @@ public class SftpFileOutputPlugin
             int taskCount,
             List<TaskReport> successTaskReports)
     {
-        SftpUtils sftpUtils = new SftpUtils(taskSource.loadTask(PluginTask.class));
-        for (TaskReport report : successTaskReports) {
-            List<Map<String, String>> moveFileList = report.get(List.class, "file_list");
-            for (Map<String, String> pairFiles : moveFileList) {
-                String temporaryFileName = pairFiles.get("temporary_filename");
-                String realFileName = pairFiles.get("real_filename");
-
-                sftpUtils.renameFile(temporaryFileName, realFileName);
-            }
-        }
-        sftpUtils.close();
+        /*
+          #37 causes permission failure while renaming remote file.
+          https://github.com/embulk/embulk-output-sftp/issues/40
+         */
+//        SftpUtils sftpUtils = new SftpUtils(taskSource.loadTask(PluginTask.class));
+//        for (TaskReport report : successTaskReports) {
+//            List<Map<String, String>> moveFileList = report.get(List.class, "file_list");
+//            for (Map<String, String> pairFiles : moveFileList) {
+//                String temporaryFileName = pairFiles.get("temporary_filename");
+//                String realFileName = pairFiles.get("real_filename");
+//
+//                sftpUtils.renameFile(temporaryFileName, realFileName);
+//            }
+//        }
+//        sftpUtils.close();
     }
 
     @Override

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: embulk-output-sftp
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Civitaspo
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-11-07 00:00:00.000000000 Z
+date: 2017-12-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -70,11 +70,11 @@ files:
 - src/test/java/org/embulk/output/sftp/TestSftpFileOutputPlugin.java
 - src/test/resources/id_rsa
 - src/test/resources/id_rsa.pub
-- classpath/commons-io-2.5.jar
+- classpath/commons-io-2.6.jar
 - classpath/commons-logging-1.2.jar
-- classpath/commons-vfs2-2.1.1660580.2.jar
-- classpath/embulk-output-sftp-0.1.3.jar
-- classpath/jsch-0.1.53.jar
+- classpath/commons-vfs2-2.2.jar
+- classpath/embulk-output-sftp-0.1.4.jar
+- classpath/jsch-0.1.54.jar
 homepage: https://github.com/embulk/embulk-output-sftp
 licenses:
 - MIT