apache / apache/parquet-java

Deprecate parquet-thrift

Open
#2,820 0 comments 0 reactions 0 assignees View on GitHub
Component: Parquet Priority: Major Type: enhancement
Dominant language
Java
Stars
3.1k
Forks
1.6k
Avg merge
3d 12h
Merged PRs (30d)
33

Description

I was in the process of updating to [the latest version of Thrift](https://github.com/apache/parquet-mr/pull/1138) (from 0.16.0 to 0.19.0). Mostly because it contains CVEs and makes the release process easier because you don't have to install Thrift from source (it is just available on homebrew etc).
 
While working on this, I ran into an issue with Elephantbird, which is using a very old version of Thrift (0.7.0). Trying to bump this I noticed that a lot of classes that we use in the tests [have](https://github.com/apache/parquet-mr/pull/1156) [ been made private](https://github.com/apache/parquet-mr/pull/1156). Therefore it is hard to test if we break anything.
 
It looks like parquet-thrift [is not used by anyone anymore](https://mvnrepository.com/artifact/org.apache.parquet/parquet-thrift). I would suggest removing this from the repository unless anyone objects.

**Reporter**: [Fokko Driesprong](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=fokko) / @Fokko
**Assignee**: [Fokko Driesprong](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=fokko) / @Fokko

**Note**: *This issue was originally created as [PARQUET-2355](https://issues.apache.org/jira/browse/PARQUET-2355). Please see the [migration documentation](https://issues.apache.org/jira/browse/PARQUET-2502) for further details.*

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.