[jira] Created: (HADOOP-3568) Don't need to use toString() on strings (code cleanup)

[Tim Halloran (JIRA)]

Don't need to use toString() on strings (code cleanup)
------------------------------------------------------

                 Key: HADOOP-3568
                 URL: https://issues.apache.org/jira/browse/HADOOP-3568
             Project: Hadoop Core
          Issue Type: Improvement
          Components: test
    Affects Versions: 0.17.0
            Reporter: Tim Halloran
            Priority: Minor


Don't need to call toString on a String type.  This occurs in several places in 
the test code.  Patches below:

patch to org.apache.hadoop.dfs.TestDFSStartupVersions

Index: .
===================================================================
--- .   (revision 8259)
+++ .   (working copy)
@@ -37,7 +37,7 @@
   private static final Log LOG = LogFactory.getLog(
                                                    
"org.apache.hadoop.dfs.TestDFSStartupVersions");
   private static Path TEST_ROOT_DIR = new Path(
-                                               
System.getProperty("test.build.data","/tmp").toString().replace(' ', '+'));
+                                               
System.getProperty("test.build.data","/tmp").replace(' ', '+'));
   private MiniDFSCluster cluster = null;
   
   /**



patch to org.apache.hadoop.mapred.lib.aggregate.TestAggregates

Index: .
===================================================================
--- .   (revision 8259)
+++ .   (working copy)
@@ -109,7 +109,7 @@
     Path outPath = new Path(OUTPUT_DIR, "part-00000");
     String outdata = TestMiniMRWithDFS.readOutput(outPath,job);
     System.out.println("full out data:");
-    System.out.println(outdata.toString());
+    System.out.println(outdata);
     outdata = outdata.substring(0, expectedOutput.toString().length());
 
     assertEquals(expectedOutput.toString(),outdata);



patch to org.apache.hadoop.mapred.NotificationTestCase

Index: .
===================================================================
--- .   (revision 8259)
+++ .   (working copy)
@@ -174,8 +174,7 @@
 
     // Hack for local FS that does not have the concept of a 'mounting point'
     if (isLocalFS()) {
-      String localPathRoot = System.getProperty("test.build.data","/tmp")
-        .toString().replace(' ', '+');;
+      String localPathRoot = 
System.getProperty("test.build.data","/tmp").replace(' ', '+');;
       inDir = new Path(localPathRoot, inDir);
       outDir = new Path(localPathRoot, outDir);
     }



patch to org.apache.hadoop.streaming.TestMultipleArchiveFiles

Index: .
===================================================================
--- .   (revision 8259)
+++ .   (working copy)
@@ -84,7 +84,7 @@
     dos.write(inputFileString.getBytes("UTF-8"));
     dos.close();
     
-    DataOutputStream out = fileSys.create(new 
Path(CACHE_ARCHIVE_1.toString()));
+    DataOutputStream out = fileSys.create(new Path(CACHE_ARCHIVE_1));
     ZipOutputStream zos = new ZipOutputStream(out);
     ZipEntry ze = new ZipEntry(CACHE_FILE_1.toString());
     zos.putNextEntry(ze);
@@ -112,7 +112,7 @@
     }
 
     return new String[] {
-      "-input", INPUT_FILE.toString(),
+      "-input", INPUT_FILE,
       "-output", OUTPUT_DIR,
       "-mapper", "xargs cat", 
       "-reducer", "cat",



-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.