Format: Do not shade slf4j-api

[asfimport]

PARQUET-369 fixed warnings from shading slf4j-api, but a consequence of shading is that the log messages from thrift for parquet-format classes are dropped. This was an accepted trade-off until PARQUET-305 changed logging in the rest of the library to SLF4J. Now that the slf4j-api is a dependency for all of Parquet except parquet-format, it no longer makes sense to suppress the format thrift logs to avoid exposing it.

This also requires PARQUET-371 because thrift 0.7.0 relies on a very old version of slf4j-api.

Reporter: Ryan Blue / @rdblue
Assignee: Julien Le Dem / @julienledem

Related issues:

• Release Parquet format 2.4.0 (blocks)
• Shading SLF4J prevents SLF4J locating org.slf4j.impl.StaticLoggerBinder (is related to)
• Logger instantiated for package org.apache.parquet may be GC-ed (is related to)
• Bumps Thrift version to 0.9.3 (requires)
• Shutdown hook in parquet-avro library corrupts data and disables logging (is required by)

PRs and other links:

• PR #50

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

[asfimport]

Michal Turek / @mixalturek:
I have just fixed the code, please review my changes and merge them.
#37

[asfimport]

Ryan Blue / @rdblue:
I'm removing this as a blocker for the 1.9.0 release since it's a format change. If we want it in, then I think we should do a format release and get that into a Parquet MR release.

[asfimport]

Ryan Blue / @rdblue:
Fixed in PR #50