Github user kostya-sh commented on the pull request:

    https://github.com/apache/spark/pull/6597#issuecomment-109142263
  
    BTW `enableLogForwarding()` should be updated as well, because name of the 
logger has been changed to `org.apache.parquet`. From parquet-mr Log:
    
        // add a default handler in case there is none
        Logger logger = Logger.getLogger(Log.class.getPackage().getName());
    
    Another problem with `enableLogForwarding()` is that it doesn't hold to 
created loggers that can be garbage collected and all configuration changes 
will be gone. From 
https://docs.oracle.com/javase/6/docs/api/java/util/logging/Logger.html 
javadocs:  _It is important to note that the Logger returned by one of the 
getLogger factory methods may be garbage collected at any time if a strong 
reference to the Logger is not kept._
    
    I've created https://issues.apache.org/jira/browse/SPARK-8122
    



---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to