[
https://issues.apache.org/jira/browse/HIVE-26691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17627724#comment-17627724
]
Ayush Saxena commented on HIVE-26691:
-------------------------------------
One problem I feel is managing the thrift home, if the versions for someone is
different upstream & downstream, this would be a bit problematic. Rest I think
if we are changing the build mechanism, we need to mention that as explicitly
in the Release Notes, some one running tests or builds on the code, his build
will fail if he doesn't have proper thrift setup.
Secondly how ofter are we upgrading Thrift, a thrift upgrade would mean all dev
to sort their dev setups to be inline with the new version right?
Does the same thing hold for protobuf also, I guess there is an explicit
profile as well for it
> Generate thrift files by default at compilation time
> ----------------------------------------------------
>
> Key: HIVE-26691
> URL: https://issues.apache.org/jira/browse/HIVE-26691
> Project: Hive
> Issue Type: Task
> Components: Thrift API
> Affects Versions: 4.0.0-alpha-2
> Reporter: Alessandro Solimando
> Priority: Major
>
> Currently Hive does not generate thrift files within the main compilation
> task ({_}mvn clean install -DskipTests{_}), but it uses a separate profile
> ({_}mvn clean install -Pthriftif -DskipTests -Dthrift.home=$thrift_path{_}),
> and thrift-generated files are generally committed in VCS.
> Other Apache projects like Parquet
> ([https://github.com/apache/parquet-mr/blob/master/parquet-thrift/pom.xml)]
> use a different approach, building all thrift files by default in the main
> compilation task.
> In general, generated files should not be part of our VCS, only the "source"
> file should be (.thrift files here).
> Including generated files in VCS is not only problematic because they are
> verbose and clog PR diffs, but they also generate a lot of conflicts (even
> when the changes over the thrift file can be merged automatically).
> The ticket proposes to move the thrift files generation at compile time,
> remove the thrift-generated files from VCS, and add them to the "ignore" list.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
