Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-6730

Bug in FileContext#copy and provide base class for FileContext tests

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.22.0
    • 0.22.0
    • fs, test
    • None
    • Incompatible change, Reviewed

    Description

      Thanks to Eli, He noticed that there is no test for FileContext#Copy operation.

      On further investigation with the help of Sanjay we found that there is bug in FileContext#checkDest.

        FileStatus dstFs = getFileStatus(dst);
          try {
            if (dstFs.isDir()) {
              if (null == srcNa
      

      FileStatus dstFs = getFileStatus(dst); should be in try...catch block.

          try {
             FileStatus dstFs = getFileStatus(dst);
             if (dstFs.isDir()) {
                if (null == srcNa
      

      Attachments

        1. HADOOP-6730.patch
          5 kB
          Ravi Phulari
        2. HADOOP-6730.4.patch
          6 kB
          Ravi Phulari
        3. HADOOP-6730.3.patch
          6 kB
          Ravi Phulari
        4. HADOOP-6730.2.patch
          7 kB
          Ravi Phulari

        Activity

          People

            raviphulari Ravi Phulari
            eli Eli Collins
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: