[jira] [Commented] (HDFS-13938) Add a missing "break" in BaseTestHttpFSWith

["Wei-Chiu Chuang (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HDFS-13938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629458#comment-16629458 ] 

Wei-Chiu Chuang commented on HDFS-13938:
----------------------------------------

+1

> Add a missing "break" in BaseTestHttpFSWith
> -------------------------------------------
>
>                 Key: HDFS-13938
>                 URL: https://issues.apache.org/jira/browse/HDFS-13938
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: httpfs
>    Affects Versions: 3.2.0
>            Reporter: Siyao Meng
>            Assignee: Siyao Meng
>            Priority: Trivial
>         Attachments: HDFS-13938.001.patch
>
>
> In BaseTestHttpFSWith:
> {code:java}
>     case DISALLOW_SNAPSHOT:
>       testDisallowSnapshot();
>       break;
>     case DISALLOW_SNAPSHOT_EXCEPTION:
>       testDisallowSnapshotException();
>       // Missed a "break" here.
>     case FILE_STATUS_ATTR:
>       testFileStatusAttr();
>       break;
> {code}
> The missing "break" won't cause any bugs though. Just the fact that testFileStatusAttr() will be run a second time after testDisallowSnapshotException() finishes. :P



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org