[
https://issues.apache.org/jira/browse/PARQUET-178?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14307773#comment-14307773
]
Julien Le Dem commented on PARQUET-178:
---------------------------------------
It may be time to revisit the inclusion of slf4j in parquet-format.
The main idea was to shade thrift in this jar so that the version of thrift we
use to read/write the metadata in the parquet file would not conflict with the
version of thrift users want with parquet-thrift.
Thrift happens to depend on slf4j so I just shaded it as well at the time so
that thrift would be totally invisible.
It sounds like we could remove it, or remove/rename META-INF/maven/org.slf4j
(which makes it *not* invisible)
> META-INF for slf4j should not be in parquet-format jar
> ------------------------------------------------------
>
> Key: PARQUET-178
> URL: https://issues.apache.org/jira/browse/PARQUET-178
> Project: Parquet
> Issue Type: Bug
> Components: parquet-format
> Affects Versions: 1.6.0rc2
> Reporter: koert kuipers
> Priority: Minor
>
> {noformat}
> $ jar tf parquet-format-2.2.0-rc1.jar | grep org\\.slf
> META-INF/maven/org.slf4j/
> META-INF/maven/org.slf4j/slf4j-api/
> META-INF/maven/org.slf4j/slf4j-api/pom.xml
> META-INF/maven/org.slf4j/slf4j-api/pom.properties
> {noformat}
> It is not clear to me why these are here. I suspect they should not be.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
