srowen commented on a change in pull request #21339: [SPARK-24287][Core] Spark
-packages option should support classifier, no-transitive, and custom conf
URL: https://github.com/apache/spark/pull/21339#discussion_r268671701
##########
File path: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala
##########
@@ -1000,33 +1001,81 @@ private[spark] object SparkSubmitUtils {
"tags_", "unsafe_")
/**
- * Represents a Maven Coordinate
- * @param groupId the groupId of the coordinate
- * @param artifactId the artifactId of the coordinate
- * @param version the version of the coordinate
+ * Represents a Artifact Coordinate to resolve
+ * @param groupId the groupId of the artifact
+ * @param artifactId the artifactId of the artifact
+ * @param version the version of the artifact
+ * @param extraParams the extra params to resolve the artifact
*/
- private[deploy] case class MavenCoordinate(groupId: String, artifactId:
String, version: String) {
- override def toString: String = s"$groupId:$artifactId:$version"
+ private[deploy] case class MavenCoordinate(groupId: String, artifactId:
String, version: String,
+ extraParams: Map[String, String] = Map()) {
+
+ def params: String = if (extraParams.isEmpty) {
+ ""
+ } else {
+ "?" + extraParams.map{ case (k, v) => (k + "=" + v) }.mkString(":")
Review comment:
This won't pass the style check I think. `"?" + extraParams.map { case (k,
v) => k + "=" + v }.mkString(":")`
But shouldn't it be `mkString("&")`?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
