Github user ash211 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/1019#discussion_r13954414
  
    --- Diff: conf/log4j.properties.template ---
    @@ -6,6 +6,6 @@ log4j.appender.console.layout=org.apache.log4j.PatternLayout
     log4j.appender.console.layout.ConversionPattern=%d{yy/MM/dd HH:mm:ss} %p 
%c{1}: %m%n
     
     # Settings to quiet third party logs that are too verbose
    -log4j.logger.org.eclipse.jetty=WARN
    --- End diff --
    
    I was assuming that this line was a prior failed attempt at silencing this 
exact issue that stuck around, so intentionally changed this logging.
    
    My change logs more verbosely for everything under `org.eclipse.jetty` that 
isn't `org.eclipse.jetty.util.component.AbstractLifeCycle` since it's lowering 
the threshold from WARN down to level of the root, which is INFO.  Even if we 
were silencing something from jetty in addition to the message this change 
addresses, it'd be better to target that class more directly rather than 
everything inside `org.eclipse.jetty`


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

Reply via email to