Bumps Thrift version to 0.9.3

[asfimport]

Thrift 0.7.0 is too old a version, and it doesn't compile on Mac. Would be nice to bump Thrift version.

Reporter: Cheng Lian / @liancheng
Assignee: Julien Le Dem / @julienledem

Related issues:

• Release Parquet format 2.4.0 (blocks)
• Format: Do not shade slf4j-api (is required by)

PRs and other links:

• PR link

_{Note: This issue was originally created as PARQUET-371. Please see the migration documentation for further details.}

[asfimport]

Ryan Blue / @rdblue:
I believe that the underlying thrift format is compatible across versions, so this should work. But, I'd rather update everything to thrift 9 instead of worrying about compatibility between the two. I don't want to have a case where we add a class that causes issues when it uses the wrong libthrift version and have that possibly sneak into a release.

[asfimport]

Cheng Lian / @liancheng:
That would be even nicer. I'll update my PR.

[asfimport]

Ryan Blue / @rdblue:
@isnotinvain, is this okay to merge? I think it's a good idea to update our thrift version, but I think you guys are the heaviest users. We'll need to check that elephantbird doesn't pull in 0.7.0 also.

[asfimport]

Ryan Blue / @rdblue:
We just talked about this in the Parquet sync and think it is a good idea. I'm marking this for inclusion in parquet-format 2.4.0. Thanks, Cheng!

[asfimport]

Ryan Blue / @rdblue:
I'm removing this as a blocker for the 1.9.0 release. This is a format change so we should release parquet-format 2.4.0 to pull it into a Parquet MR release.

[asfimport]

Ryan Blue / @rdblue:
This PR was merged.