Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-13938

Add a missing "break" in BaseTestHttpFSWith

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2.0
    • Fix Version/s: 3.2.0
    • Component/s: httpfs, test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      In BaseTestHttpFSWith:

          case DISALLOW_SNAPSHOT:
            testDisallowSnapshot();
            break;
          case DISALLOW_SNAPSHOT_EXCEPTION:
            testDisallowSnapshotException();
            // Missed a "break" here.
          case FILE_STATUS_ATTR:
            testFileStatusAttr();
            break;
      

      The missing "break" won't cause any bugs though. Just the fact that testFileStatusAttr() will be run a second time after testDisallowSnapshotException() finishes.

        Attachments

        1. HDFS-13938.001.patch
          0.7 kB
          Siyao Meng
        2. HDFS-13938.001.patch
          0.7 kB
          Siyao Meng
        3. HDFS-13938.002.patch
          0.7 kB
          Yiqun Lin

          Issue Links

            Activity

              People

              • Assignee:
                smeng Siyao Meng
                Reporter:
                smeng Siyao Meng
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: