[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread WangTaoTheTonic
GitHub user WangTaoTheTonic opened a pull request: https://github.com/apache/spark/pull/2445 [SPARK-3589][Minor]remove redundant code You can merge this pull request into a Git repository by running: $ git pull https://github.com/WangTaoTheTonic/spark removeRedundant

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2445#issuecomment-56026443 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20538/consoleFull) for PR 2445 at commit

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2445#issuecomment-56032317 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20538/consoleFull) for PR 2445 at commit

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/2445#discussion_r17747811 --- Diff: bin/spark-class --- @@ -169,7 +169,6 @@ if [ -n $SPARK_SUBMIT_BOOTSTRAP_DRIVER ]; then # This is used only if the properties file actually

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2445#issuecomment-56087106 Thanks - I merged this but I removed one of the changes which was incorrect. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2445 --- 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

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/2445#discussion_r17749059 --- Diff: bin/spark-class --- @@ -169,7 +169,6 @@ if [ -n $SPARK_SUBMIT_BOOTSTRAP_DRIVER ]; then # This is used only if the properties file actually

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/2445#discussion_r17765294 --- Diff: bin/spark-class --- @@ -169,7 +169,6 @@ if [ -n $SPARK_SUBMIT_BOOTSTRAP_DRIVER ]; then # This is used only if the properties file

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/2445#discussion_r17765764 --- Diff: bin/spark-class --- @@ -169,7 +169,6 @@ if [ -n $SPARK_SUBMIT_BOOTSTRAP_DRIVER ]; then # This is used only if the properties file actually

[GitHub] spark pull request: [SPARK-3589][Minor]remove redundant code

2014-09-18 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/2445#discussion_r17766944 --- Diff: bin/spark-class --- @@ -169,7 +169,6 @@ if [ -n $SPARK_SUBMIT_BOOTSTRAP_DRIVER ]; then # This is used only if the properties file