[GitHub] spark pull request: [SPARK-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/8835#discussion_r40132036 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUDFs.scala --- @@ -342,51 +348,57 @@ case class BatchPythonEvaluation(udf:

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40133609 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142396754 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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142397243 [Test build #42851 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42851/consoleFull) for PR 8843 at commit

[GitHub] spark pull request: [SPARK-10300] [BUILD] [TESTS] Add support for ...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8775#issuecomment-142396888 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10737] [SQL] When using UnsafeRows, Sor...

2015-09-22 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8854#issuecomment-142397298 @JoshRosen Can you also take a look? --- 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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142401254 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

[GitHub] spark pull request: [SPARK-9883][MLlib] Distance to each kmean clu...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8849#issuecomment-142401217 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142404458 [Test build #42855 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42855/consoleFull) for PR 8860 at commit

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142404579 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10737] [SQL] When using UnsafeRows, Sor...

2015-09-22 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8854#discussion_r40117487 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala --- @@ -82,6 +79,28 @@ case class SortMergeJoin(

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-22 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-142361045 LGTM; the duplicate event issue is very minor and doesn't happen with the default config (where connections per peer is 1), so not super critical to fix now. --- If

[GitHub] spark pull request: [SPARK-10515] When killing executor, the pendi...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8668#issuecomment-142363716 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10593] [SQL] fix resolve output of Gene...

2015-09-22 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8755#discussion_r40119174 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -378,6 +378,22 @@ class Analyzer( val

[GitHub] spark pull request: [SPARK-10515] When killing executor, the pendi...

2015-09-22 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/8668#discussion_r40121163 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala --- @@ -147,6 +150,10 @@ class

[GitHub] spark pull request: [SPARK-10515] When killing executor, the pendi...

2015-09-22 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/8668#discussion_r40121097 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala --- @@ -66,6 +66,9 @@ class

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8870#issuecomment-142370370 ok to test --- 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

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8870#issuecomment-142372122 [Test build #42845 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42845/consoleFull) for PR 8870 at commit

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8870#issuecomment-142379767 [Test build #42845 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42845/console) for PR 8870 at commit

[GitHub] spark pull request: [SPARK-10704] Rename HashShuffleReader to Bloc...

2015-09-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8825#issuecomment-142383435 LGTM. merging in master. --- 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

[GitHub] spark pull request: [SPARK-10740][SQL] handle nondeterministic exp...

2015-09-22 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8858#discussion_r40129413 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -129,34 +129,63 @@ object SetOperationPushDown extends

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142387000 @zhichao-li I further special cased `LazySimpleSerDe`, so that we always use `TextRecordReader`/`TextRecordWriter` together with it, and uses can customize field

[GitHub] spark pull request: [SPARK-10652][SPARK-10742][Streaming] Set mean...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8791#issuecomment-142390475 [Test build #42849 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42849/consoleFull) for PR 8791 at commit

[GitHub] spark pull request: [SPARK-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/8835#discussion_r40131699 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUDFs.scala --- @@ -342,51 +348,57 @@ case class BatchPythonEvaluation(udf:

[GitHub] spark pull request: [SPARK-10672] [SQL] Do not fail when we cannot...

2015-09-22 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8824#issuecomment-142392010 LGTM --- 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

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40125900 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -167,10 +169,18 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40125908 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -167,10 +169,18 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40125943 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -302,7 +320,44 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40125898 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -60,12 +60,14 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40125924 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -235,10 +245,18 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40127437 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142383961 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142383987 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142386815 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142386846 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

[GitHub] spark pull request: [SPARK-10740][SQL] handle nondeterministic exp...

2015-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8858 --- 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-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40131267 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: [SPARK-10381] Fix mixup of taskAttemptNumber &...

2015-09-22 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8790#issuecomment-142392584 test this please --- 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

[GitHub] spark pull request: [SPARK-10381] Fix mixup of taskAttemptNumber &...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8790#issuecomment-142395999 [Test build #42850 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42850/consoleFull) for PR 8790 at commit

[GitHub] spark pull request: [SPARK-10672] [SQL] Do not fail when we cannot...

2015-09-22 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8824#issuecomment-142397682 Let me see if I can add a test. --- 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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8843#discussion_r40135061 --- Diff: core/src/main/scala/org/apache/spark/rdd/ReliableRDDCheckpointData.scala --- @@ -101,7 +101,9 @@ private[spark] object ReliableRDDCheckpointData {

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142403967 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

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40124054 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -90,6 +108,12 @@ private[regression] trait

[GitHub] spark pull request: [SPARK-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8836#discussion_r40126117 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -302,7 +320,44 @@ class AFTSurvivalRegressionSuite

[GitHub] spark pull request: [SPARK-10740][SQL] handle nondeterministic exp...

2015-09-22 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8858#discussion_r40127014 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -916,4 +916,43 @@ class DataFrameSuite extends QueryTest with

[GitHub] spark pull request: [SPARK-10652][SPARK-10742][Streaming] Set mean...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8791#issuecomment-142390213 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

[GitHub] spark pull request: [SPARK-10652][SPARK-10742][Streaming] Set mean...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8791#issuecomment-142390243 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142390287 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142390289 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/8835#discussion_r40131745 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUDFs.scala --- @@ -342,51 +348,57 @@ case class BatchPythonEvaluation(udf:

[GitHub] spark pull request: [SPARK-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/8835#discussion_r40133051 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUDFs.scala --- @@ -329,7 +329,13 @@ case class EvaluatePython( /** * ::

[GitHub] spark pull request: [SPARK-10300] [BUILD] [TESTS] Add support for ...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8775#issuecomment-142396548 [Test build #42844 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42844/console) for PR 8775 at commit

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142396675 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

[GitHub] spark pull request: [SPARK-10300] [BUILD] [TESTS] Add support for ...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8775#issuecomment-142396883 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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142401220 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

[GitHub] spark pull request: [SPARK-9883][MLlib] Distance to each kmean clu...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8849#issuecomment-142401253 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

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8870#issuecomment-142380009 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

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8870#issuecomment-142380013 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10704] Rename HashShuffleReader to Bloc...

2015-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8825 --- 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-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142383691 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142384704 [Test build #42846 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42846/consoleFull) for PR 8860 at commit

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8843#discussion_r40128458 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillReader.java --- @@ -24,12 +24,15 @@ import

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142403371 [Test build #42853 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42853/consoleFull) for PR 8843 at commit

[GitHub] spark pull request: [SPARK-9883][MLlib] Distance to each kmean clu...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8849#issuecomment-142403265 [Test build #42854 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42854/consoleFull) for PR 8849 at commit

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40126866 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142383516 [Test build #42842 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42842/console) for PR 8843 at commit

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-142383683 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

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40129611 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: [SPARK-10760] [SparkR] SparkR glm: the documen...

2015-09-22 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/8870#discussion_r40130655 --- Diff: R/pkg/R/mllib.R --- @@ -41,7 +41,8 @@ setClass("PipelineModel", representation(model = "jobj")) #' sqlContext <- sparkRSQL.init(sc) #'

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142389545 [Test build #42848 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42848/consoleFull) for PR 8860 at commit

[GitHub] spark pull request: [SPARK-10740][SQL] handle nondeterministic exp...

2015-09-22 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8858#issuecomment-142388012 LGTM. Merging to master and branch 1.5. --- 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142387961 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142387993 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

[GitHub] spark pull request: [SPARK-10737] [SQL] When using UnsafeRows, Sor...

2015-09-22 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8854#issuecomment-142392425 LGTM --- 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

[GitHub] spark pull request: [SPARK-10381] Fix mixup of taskAttemptNumber &...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8790#issuecomment-142393514 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

[GitHub] spark pull request: [SPARK-10381] Fix mixup of taskAttemptNumber &...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8790#issuecomment-142393488 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

[GitHub] spark pull request: [SPARK-9883][MLlib] Distance to each kmean clu...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8849#issuecomment-142402703 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

[GitHub] spark pull request: [SPARK-9883][MLlib] Distance to each kmean clu...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8849#issuecomment-142402761 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

[GitHub] spark pull request: [SPARK-10310] [SQL] Fixes script transformatio...

2015-09-22 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8860#issuecomment-142403510 test this please --- 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

[GitHub] spark pull request: [SPARK-10656][SQL]fix selection fails when a c...

2015-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8811#issuecomment-142188133 [Test build #42814 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42814/console) for PR 8811 at commit

[GitHub] spark pull request: [SPARK-9718][ML] linear regression training su...

2015-09-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/8564#discussion_r40055543 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -298,11 +298,7 @@ class LinearRegressionModel private[ml] (

[GitHub] spark pull request: [SPARK-8334] [SQL] Binary logical plan should ...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6790#issuecomment-142189260 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

[GitHub] spark pull request: [SPARK-9821][PYSPARK] pyspark-reduceByKey-shou...

2015-09-22 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/8569#issuecomment-142189285 @davies this should now work in the other places which are using cogroup under the hood. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SQL][Minor] map -> foreach.

2015-09-22 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/8862 [SQL][Minor] map -> foreach. DataFrame.explain should use foreach to print the explain content. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-9774][ML][PYSPARK] Add python api for m...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8214#issuecomment-142193435 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

[GitHub] spark pull request: [SPARK-9774][ML][PYSPARK] Add python api for m...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8214#issuecomment-142193376 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

[GitHub] spark pull request: [SPARK-10577] [PySpark] DataFrame hint for bro...

2015-09-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8801#issuecomment-142194415 Thanks - I've merged 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

[GitHub] spark pull request: [SQL][Minor] map -> foreach.

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8862#issuecomment-142194504 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

[GitHub] spark pull request: [SPARK-10750] [ML] ML Param validate should pr...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8863#issuecomment-142194452 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

[GitHub] spark pull request: [SQL][Minor] map -> foreach.

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8862#issuecomment-142194507 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10750] [ML] ML Param validate should pr...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8863#issuecomment-142194458 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

[GitHub] spark pull request: [SPARK-10419] [SQL] Adding SQLServer support f...

2015-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8575 --- 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-10446][SQL] Support to specify join typ...

2015-09-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8600#issuecomment-142195275 Thanks - I've merged 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

[GitHub] spark pull request: [SPARK-10458] [Spark Core] Added isStopped() m...

2015-09-22 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8749#discussion_r40057454 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -264,6 +264,10 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-10458] [Spark Core] Added isStopped() m...

2015-09-22 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8749#issuecomment-142198411 Merging into master. --- 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

[GitHub] spark pull request: [SPARK-10695] [Documentation][Mesos] Fixing in...

2015-09-22 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8816#issuecomment-142199902 Oops that looked wrong. Merging this into master 1.5 thanks for catching this. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-10750] [ML] ML Param validate should pr...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8863#issuecomment-142199703 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

[GitHub] spark pull request: [SPARK-9774][ML][PYSPARK] Add python api for m...

2015-09-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/8214#discussion_r40057949 --- Diff: python/pyspark/ml/param/_shared_params_code_gen.py --- @@ -47,7 +47,7 @@ def _gen_param_header(name, doc, defaultValueStr): """

[GitHub] spark pull request: [SPARK-10656][SQL]fix selection fails when a c...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8811#issuecomment-142188195 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10656][SQL]fix selection fails when a c...

2015-09-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8811#issuecomment-142188194 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

  1   2   3   4   5   6   7   8   >