Github user AndreSchumacher commented on the pull request:
https://github.com/apache/spark/pull/325#issuecomment-39701612
@witgo Good idea to move the forwarding setup out of there but in the
process it seems that edd96307a10fe7e869bd39afb0cdbbc3a8f46c4d broke the
forwarding again.
Also: I think it is confusing to set the log level to null (default).
Parquet hides most log statements behind ```if (level >= WARN)``` etc. and
relies on the compiler to remove these for performance reasons. So we should
just do the same as parquet.Log does and do
```parquetLogger.setLevel(parquet.Log.LEVEL)``` so we won't confuse users. Note
that the actual level can be still set via Log4J (assuming it's not finer than
that).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
