luhenry commented on pull request #32415: URL: https://github.com/apache/spark/pull/32415#issuecomment-839056925
> So the `core` artifact is no longer part of the transitive deps? I would think `all` needs it, still, but, not sure. `core` is still part of the transitive dependencies through `breeze`. If we add `all`, then we made sure that it is on the classpath, and that `breeze` has access to the accelerated implementation in `com.github.fommil.netlib:all`. And `all` itself references `core`, so not referencing `core` isn't an issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
