PARQUET-2344: Bump to Thrift 0.19.0

[Fokko]

Make sure you have checked all steps below.

This requires a new release of parquet-format, but it looks like @wgtmac is on top of that: apache/parquet-format#197 (comment)

Jira

• My PR addresses the following Parquet Jira issues and references them in the PR title. For example, "PARQUET-1234: My Parquet PR"
  • https://issues.apache.org/jira/browse/PARQUET-XXX
  • In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

• My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

• My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":
  1. Subject is separated from body by a blank line
  2. Subject is limited to 50 characters (not including Jira issue reference)
  3. Subject does not end with a period
  4. Subject uses the imperative mood ("add", not "adding")
  5. Body wraps at 72 characters
  6. Body explains "what" and "why", not "how"

Documentation

• In case of new functionality, my PR adds documentation that describes how to use it.
  • All the public functions and the classes in the PR contain Javadoc that explain what it does

[wgtmac]

Do you need any additional fix to parquet-format before the v2.10 release? @Fokko

[Fokko]

@wgtmac No just the release, thanks for asking :)

[wgtmac]

@wgtmac No just the release, thanks for asking :)

Thanks for confirmation!

[wgtmac]

@Fokko Do you want to include parquet-format 2.10.0 adoption in this PR or create a separate one? IMO, there are several PRs waiting for 2.10.0 to rebase so it would be good to check that in first?

[Fokko]

@wgtmac Thanks for splitting out the format upgrade. Always a good idea to make PRs smaller.

I finally fixed all the tests, and this looks good to go to me 👍

[wgtmac]

Why mixing UUID and ENUM in this case?

[Fokko]

It looks like there are still enums with field-id 16. It can be that they are using an old Java SDK. This was a reliable way do differentiate between them.

[wgtmac]

Why do we need to ignore this?

[wgtmac]

Where do we need this?

[wgtmac]

Do you want to merge this?

[Fokko]

@wgtmac Yes, I'd love to. Thanks 🙌