Github user prb commented on the pull request:
https://github.com/apache/incubator-spark/pull/573#issuecomment-34715973
The infinite loop was caused by the fact that the list of appenders is
*always* empty when the slf4j mock implementation of log4j is in place.
Github user prb commented on the pull request:
https://github.com/apache/incubator-spark/pull/570#issuecomment-34606733
@pwendell I think @srowen's changes are actually cleaning up some different
loose ends around jcl versus log4j, and what he's talking about being incumbe
