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

Ashutosh Chauhan commented on HIVE-5671:
----------------------------------------

Reason for checking in {{OrcProto.java}} is same as checking in thrift 
generated code which is if not checked in then that implies devs need to have 
thrift compiler (for thrift generated code)  and protobuf compiler (for 
protobuf generated code) to be installed on their machine. In the past, many 
devs have felt that installing thrift compiler or protobuf compiler is 
complicated enough that if possible we should avoid the burden of having it 
installed. Also, having this requirement will be impediment for new hive devs.

> Generated src OrcProto.java shouldn't be in the source control
> --------------------------------------------------------------
>
>                 Key: HIVE-5671
>                 URL: https://issues.apache.org/jira/browse/HIVE-5671
>             Project: Hive
>          Issue Type: Improvement
>          Components: Build Infrastructure
>    Affects Versions: 0.12.0
>            Reporter: Xuefu Zhang
>            Assignee: Xuefu Zhang
>
> orc_proto.proto generates OrcProto.java, which unfortunately made its way to 
> source control, so changing the .proto file requires regenerating the .java 
> file and checking it in again. This is unnecessary.
> Also, the code generation should be part of the build process.



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

Reply via email to