Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17150#discussion_r128728089
--- Diff: pom.xml ---
@@ -498,7 +498,7 @@
<groupId>org.slf4j</groupId>
<artifactId>jcl-over-slf4j</artifactId>
<version>${slf4j.version}</version>
- <!-- <scope>runtime</scope> --> <!-- more correct, but scalac
2.10.3 doesn't like it -->
+ <scope>runtime</scope>
--- End diff --
@HyukjinKwon it was this change. I think we need to retain it to keep
commons-logging (well, the SLF4J shim for it) on the classpath. SBT isn't
including it in the build somehow even though it's `runtime` scope, which is
really correct. I think we've seen this with the Maven-SBT plugin before.
---
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]
