[ 
https://issues.apache.org/jira/browse/HIVE-1123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12828969#action_12828969
 ] 

Zheng Shao commented on HIVE-1123:
----------------------------------

Another question. Is it recommended NOT to have the curly braces for a case 
statement?
{code}
       switch (type) {
-      case BYTE: {
+      case BYTE: 
         tbIn.readByte((ByteWritable) w);
         break;
-      }
-      case BOOL: {
+      case BOOL: 
         tbIn.readBoolean((BooleanWritable) w);
         break;
-      }
{code}


> Checkstyle fixes
> ----------------
>
>                 Key: HIVE-1123
>                 URL: https://issues.apache.org/jira/browse/HIVE-1123
>             Project: Hadoop Hive
>          Issue Type: Task
>            Reporter: Carl Steinbach
>            Assignee: Carl Steinbach
>         Attachments: HIVE-1123.cli.patch, HIVE-1123.common.patch, 
> HIVE-1123.contrib.patch, HIVE-1123.hwi.patch, HIVE-1123.jdbc.patch, 
> HIVE-1123.metastore.patch, HIVE-1123.ql.patch, HIVE-1123.serde.patch, 
> HIVE-1123.service.patch, HIVE-1123.shims.patch
>
>
> Fix checkstyle errors.

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

Reply via email to