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

Prasad Mujumdar commented on HIVE-5780:
---------------------------------------

[~brocknoland] Thanks for taking a look. The patch is just fixing the .thrift 
file which will not make a difference to the build.  The thrift generated code 
does contain the right declaration, just that it's missing from the IDL. Looks 
like some rebase/merge issue.
I guess the guidelines we are following for thrift changes is to include the 
IDL changes only as part of the patch, not the generated code. Please let me 
know if you want me to upload the generated code as well.


> Add the missing declaration of HIVE_CLI_SERVICE_PROTOCOL_V4 in 
> TCLIService.thrift
> ---------------------------------------------------------------------------------
>
>                 Key: HIVE-5780
>                 URL: https://issues.apache.org/jira/browse/HIVE-5780
>             Project: Hive
>          Issue Type: Bug
>          Components: HiveServer2
>    Affects Versions: 0.13.0
>            Reporter: Prasad Mujumdar
>            Assignee: Prasad Mujumdar
>         Attachments: HIVE-5780.1.patch
>
>
> TCLIService.thrift  is updated as part of HIVE-5355. The new enum 
>  HIVE_CLI_SERVICE_PROTOCOL_V4 is referred in the file, but the declaration is 
> missing.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to