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 incumbent on the integrator (e.g., me) is performing the exclusion of log4j/sl4j-log4j. Otherwise, I think his changes are preserving the status quo. So your PR #560 resolves the stack overflow issue and this PR further tidies up logging wrt slf4j.
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy logg... srowen
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... pwendell
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... prb
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... pwendell
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... pwendell
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... pwendell
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... srowen
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... srowen
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... AmplabJenkins
- [GitHub] incubator-spark pull request: SPARK-1071: Tidy... pwendell