You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by brkyvz <gi...@git.apache.org> on 2015/04/29 23:54:22 UTC

[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

GitHub user brkyvz opened a pull request:

    https://github.com/apache/spark/pull/5790

    [SPARK-7224] added mock repository generator for --packages tests

    This patch contains an `IvyTestUtils` file, which dynamically generates jars and pom files to test the `--packages` feature without having to rely on the internet, and Maven Central.
    
    cc @pwendell I know that there existed Util functions to create Jars and stuff already, but they didn't really serve my purposes as they appended random prefixes that was breaking things.
    
    I also added the local repository tests. Notice that they work without passing the `repo` to `resolveMavenCoordinates`.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/brkyvz/spark maven-utils

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/5790.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #5790
    
----
commit 7476d06c3dfa46f646d7d1bf5d142760e67e81d0
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-04-29T21:46:36Z

    added mock repository generator

commit 172dfef33c35570b61e7afee498d0ea9b2a17f3b
Author: Burak Yavuz <br...@gmail.com>
Date:   2015-04-29T21:53:56Z

    use Ivy format

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97624117
  
    Merged build finished. Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97602883
  
    Merged build finished. Test FAILed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97827136
  
      [Test build #31425 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31425/consoleFull) for   PR 5790 at commit [`3ec79b7`](https://github.com/apache/spark/commit/3ec79b725d7c7cb587cdf86c39c6832e1748b36d).


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97870497
  
    Merged build finished. Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97600163
  
     Merged build triggered.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97624118
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31348/
    Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97600192
  
    Merged build started.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by pwendell <gi...@git.apache.org>.
Github user pwendell commented on a diff in the pull request:

    https://github.com/apache/spark/pull/5790#discussion_r29386023
  
    --- Diff: core/src/test/scala/org/apache/spark/deploy/IvyTestUtils.scala ---
    @@ -0,0 +1,278 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *    http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +
    +package org.apache.spark.deploy
    +
    +import java.io.{File, FileInputStream, FileOutputStream}
    +import java.nio.file.{Files, Path}
    +import java.util.jar.{JarEntry, JarOutputStream}
    +import javax.tools.{JavaFileObject, SimpleJavaFileObject, StandardLocation, ToolProvider}
    +
    +import scala.collection.JavaConversions._
    +
    +import com.google.common.io.ByteStreams
    +
    +import org.apache.commons.io.FileUtils
    +
    +import org.apache.spark.deploy.SparkSubmitUtils.MavenCoordinate
    +
    +private[deploy] object IvyTestUtils {
    +
    +  /**
    +   * Create the path for the jar and pom from the maven coordinate. Extension should be `jar`
    +   * or `pom`.
    +   */
    +  private def pathFromCoordinate(
    +      artifact: MavenCoordinate,
    +      prefix: Path,
    +      ext: String,
    +      useIvyLayout: Boolean): Path = {
    +    val groupDirs = artifact.groupId.replace(".", File.separator)
    +    val artifactDirs = artifact.artifactId
    +    val artifactPath =
    +      if (!useIvyLayout) {
    +        Seq(groupDirs, artifactDirs, artifact.version).mkString(File.separator)
    +      } else {
    +        Seq(groupDirs, artifactDirs, artifact.version, ext + "s").mkString(File.separator)
    +      }
    +    new File(prefix.toFile, artifactPath).toPath
    +  }
    +
    +  private def artifactName(artifact: MavenCoordinate, ext: String = ".jar"): String = {
    +    s"${artifact.artifactId}-${artifact.version}$ext"
    +  }
    +
    +  /** Write the contents to a file to the supplied directory. */
    +  private def writeFile(dir: File, fileName: String, contents: String): File = {
    +    val outputFile = new File(dir, fileName)
    +    val outputStream = new FileOutputStream(outputFile)
    +    outputStream.write(contents.toCharArray.map(_.toByte))
    +    outputStream.close()
    +    outputFile
    +  }
    +
    +  /** Create an example Python file. */
    +  private def createPythonFile(dir: File): File = {
    +    val contents =
    +      """def myfunc(x):
    +        |   return x + 1
    +      """.stripMargin
    +    writeFile(dir, "mylib.py", contents)
    +  }
    +
    +  /** Compilable Java Source File */
    +  private class JavaSourceFromString(val file: File, val code: String)
    +    extends SimpleJavaFileObject(file.toURI, JavaFileObject.Kind.SOURCE) {
    +    override def getCharContent(ignoreEncodingErrors: Boolean) = code
    +  }
    +
    +  /** Create a simple testable Class. */
    +  private def createJavaClass(dir: File, className: String, packageName: String): File = {
    --- End diff --
    
    I think this is duplicating some of the logic in:
    https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/TestUtils.scala#L114


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by brkyvz <gi...@git.apache.org>.
Github user brkyvz commented on a diff in the pull request:

    https://github.com/apache/spark/pull/5790#discussion_r29387481
  
    --- Diff: core/src/test/scala/org/apache/spark/deploy/IvyTestUtils.scala ---
    @@ -0,0 +1,278 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *    http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +
    +package org.apache.spark.deploy
    +
    +import java.io.{File, FileInputStream, FileOutputStream}
    +import java.nio.file.{Files, Path}
    +import java.util.jar.{JarEntry, JarOutputStream}
    +import javax.tools.{JavaFileObject, SimpleJavaFileObject, StandardLocation, ToolProvider}
    +
    +import scala.collection.JavaConversions._
    +
    +import com.google.common.io.ByteStreams
    +
    +import org.apache.commons.io.FileUtils
    +
    +import org.apache.spark.deploy.SparkSubmitUtils.MavenCoordinate
    +
    +private[deploy] object IvyTestUtils {
    +
    +  /**
    +   * Create the path for the jar and pom from the maven coordinate. Extension should be `jar`
    +   * or `pom`.
    +   */
    +  private def pathFromCoordinate(
    +      artifact: MavenCoordinate,
    +      prefix: Path,
    +      ext: String,
    +      useIvyLayout: Boolean): Path = {
    +    val groupDirs = artifact.groupId.replace(".", File.separator)
    +    val artifactDirs = artifact.artifactId
    +    val artifactPath =
    +      if (!useIvyLayout) {
    +        Seq(groupDirs, artifactDirs, artifact.version).mkString(File.separator)
    +      } else {
    +        Seq(groupDirs, artifactDirs, artifact.version, ext + "s").mkString(File.separator)
    +      }
    +    new File(prefix.toFile, artifactPath).toPath
    +  }
    +
    +  private def artifactName(artifact: MavenCoordinate, ext: String = ".jar"): String = {
    +    s"${artifact.artifactId}-${artifact.version}$ext"
    +  }
    +
    +  /** Write the contents to a file to the supplied directory. */
    +  private def writeFile(dir: File, fileName: String, contents: String): File = {
    +    val outputFile = new File(dir, fileName)
    +    val outputStream = new FileOutputStream(outputFile)
    +    outputStream.write(contents.toCharArray.map(_.toByte))
    +    outputStream.close()
    +    outputFile
    +  }
    +
    +  /** Create an example Python file. */
    +  private def createPythonFile(dir: File): File = {
    +    val contents =
    +      """def myfunc(x):
    +        |   return x + 1
    +      """.stripMargin
    +    writeFile(dir, "mylib.py", contents)
    +  }
    +
    +  /** Compilable Java Source File */
    +  private class JavaSourceFromString(val file: File, val code: String)
    +    extends SimpleJavaFileObject(file.toURI, JavaFileObject.Kind.SOURCE) {
    +    override def getCharContent(ignoreEncodingErrors: Boolean) = code
    +  }
    +
    +  /** Create a simple testable Class. */
    +  private def createJavaClass(dir: File, className: String, packageName: String): File = {
    --- End diff --
    
    oh yeah, forgot to remove that. Thanks!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/spark/pull/5790


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97826823
  
    Merged build started.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97624113
  
      [Test build #31348 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31348/consoleFull) for   PR 5790 at commit [`a39151b`](https://github.com/apache/spark/commit/a39151bbe87fd773083b3a5d23050f0958462862).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds no public classes.
     * This patch does not change any dependencies.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97870498
  
    Test PASSed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31425/
    Test PASSed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97613398
  
    Merged build started.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97870471
  
      [Test build #31425 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31425/consoleFull) for   PR 5790 at commit [`3ec79b7`](https://github.com/apache/spark/commit/3ec79b725d7c7cb587cdf86c39c6832e1748b36d).
     * This patch **passes all tests**.
     * This patch merges cleanly.
     * This patch adds the following public classes _(experimental)_:
      * `      "public class " + className + extendsText + " implements java.io.Serializable `
    
     * This patch does not change any dependencies.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by pwendell <gi...@git.apache.org>.
Github user pwendell commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97884916
  
    Looks good - thanks I can merge this.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97602885
  
    Test FAILed.
    Refer to this link for build results (access rights to CI server needed): 
    https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31336/
    Test FAILed.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97826789
  
     Merged build triggered.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by pwendell <gi...@git.apache.org>.
Github user pwendell commented on a diff in the pull request:

    https://github.com/apache/spark/pull/5790#discussion_r29409553
  
    --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
    @@ -731,6 +734,10 @@ private[deploy] object SparkSubmitUtils {
         }
       }
     
    +  /** Path of the local Maven cache. */
    +  def m2Path: JavaPath = new File(System.getProperty("user.home"),
    --- End diff --
    
    make this private spark


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97613488
  
      [Test build #31348 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31348/consoleFull) for   PR 5790 at commit [`a39151b`](https://github.com/apache/spark/commit/a39151bbe87fd773083b3a5d23050f0958462862).


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] spark pull request: [SPARK-7224] added mock repository generator f...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the pull request:

    https://github.com/apache/spark/pull/5790#issuecomment-97613387
  
     Merged build triggered.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org