[HDFS-13938] Add a missing "break" in BaseTestHttpFSWith

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.2.0
• Fix Version/s: 3.2.0
• Component/s: httpfs, test
• Hadoop Flags: Reviewed
• Assignee: Siyao Meng
• Reporter: Siyao Meng
• Created: 24/Sep/18 23:56
• Updated: 02/Oct/19 17:14
• Resolved: 28/Sep/18 06:46

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.

Issue Links

• is caused by: HDFS-13876 HttpFS: Implement ALLOWSNAPSHOT, DISALLOWSNAPSHOT