Author: joyce
Date: Wed Nov 18 19:45:27 2015
New Revision: 1715046

URL: http://svn.apache.org/viewvc?rev=1715046&view=rev
Log:
NUTCH-2173 - Fix String.join issue in FileDumper

Modified:
    nutch/trunk/CHANGES.txt
    nutch/trunk/src/java/org/apache/nutch/tools/FileDumper.java

Modified: nutch/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/nutch/trunk/CHANGES.txt?rev=1715046&r1=1715045&r2=1715046&view=diff
==============================================================================
--- nutch/trunk/CHANGES.txt (original)
+++ nutch/trunk/CHANGES.txt Wed Nov 18 19:45:27 2015
@@ -3,6 +3,8 @@ Nutch Change Log
 Nutch 1.11 Release 25/10/2015 (dd/mm/yyyy)
 Release Report: http://s.apache.org/nutch11
 
+* NUTCH-2173 String.join in FileDumper breaks the build (joyce)
+
 * NUTCH-2166 Add reverse URL format to dump tool (joyce)
 
 * NUTCH-2157 Addressing Miredot REST API Warnings (Sujen Shah)

Modified: nutch/trunk/src/java/org/apache/nutch/tools/FileDumper.java
URL: 
http://svn.apache.org/viewvc/nutch/trunk/src/java/org/apache/nutch/tools/FileDumper.java?rev=1715046&r1=1715045&r2=1715046&view=diff
==============================================================================
--- nutch/trunk/src/java/org/apache/nutch/tools/FileDumper.java (original)
+++ nutch/trunk/src/java/org/apache/nutch/tools/FileDumper.java Wed Nov 18 
19:45:27 2015
@@ -256,12 +256,12 @@ public class FileDumper {
                       reversedURL[reversedURL.length - 1] += "_file";
                     }
 
-                    String reversedURLPath = String.join("/", reversedURL);
+                    String reversedURLPath = 
org.apache.commons.lang3.StringUtils.join(reversedURL, "/");
                     outputFullPath = String.format("%s/%s", fullDir, 
reversedURLPath);
                     
                     // We'll drop the trailing file name and create the nested 
structure if it doesn't already exist.
                     String[] splitPath = outputFullPath.split("/");
-                    File fullOutputDir = new File(String.join("/", 
Arrays.asList(splitPath).subList(0, splitPath.length - 1)));
+                    File fullOutputDir = new 
File(org.apache.commons.lang3.StringUtils.join(Arrays.copyOf(splitPath, 
splitPath.length - 1), "/"));
 
                     if (!fullOutputDir.exists()) {
                       fullOutputDir.mkdirs();


Reply via email to