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

Prasanth Jayachandran commented on HIVE-14854:
----------------------------------------------

nit: typo in "CoreClusteType"
enum values all capital case?

Other than that lgtm, +1

> Add a core cluster type to QTestUtil
> ------------------------------------
>
>                 Key: HIVE-14854
>                 URL: https://issues.apache.org/jira/browse/HIVE-14854
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: HIVE-14854.01.patch, HIVE-14854.02.patch
>
>
> Follow up to HIVE-14824. There's tez, tez_local, llap, llap_local - all of 
> which are of a single type, similaryl spark, sparkOnYarn, and none,mr. 
> Introducing a core cluster type to make a bunch of conditional checks simpler.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to