[jira] [Commented] (PARQUET-127) Use published Thrift plugin/artifact

["Neal Sidhwaney (JIRA)" <ji...@apache.org>]

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

Neal Sidhwaney commented on PARQUET-127:
----------------------------------------

I am surprised that no one is running into an error I keep running into, which is if the user has a different version than 0.7 installed, incompatible thrift definitions will be generated that aren't usable with libthrift 0.7 (which appears to be the one used by default during the build, since it looks to me it's being pulled in via the Twitter elephant bird library, which explicitly depends on thrift-0.7.0) 

I.E. I have 0.9.2 installed, and the java generator for 0.9.2 generates classes that aren't present in libthrift 0.7.0.  If I force 0.9.2 in the root pom of the parquet-mr project, I get a different build error.


> Use published Thrift plugin/artifact
> ------------------------------------
>
>                 Key: PARQUET-127
>                 URL: https://issues.apache.org/jira/browse/PARQUET-127
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-mr
>            Reporter: Frank Austin Nothaft
>            Assignee: Jake Farrell
>            Priority: Minor
>
> Thrift is now publishing the [Thrift tools|https://issues.apache.org/jira/browse/THRIFT-1536] to Maven Central. We should move Parquet to depend on the Maven Central artifact instead of the artifact published in the Twitter repo. Also, we should depend on the Maven libthrift artifact instead of requiring users to have libthrift installed.



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