[GitHub] spark pull request #17733: [SPARK-20425][SQL] Support a vertical display mod...

2017-04-25 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/17733#discussion_r113370773 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -277,43 +279,73 @@ class Dataset[T] private[sql]( val sb = new

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17758 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76167/ Test PASSed. ---

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17758 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

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17758 **[Test build #76167 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76167/testReport)** for PR 17758 at commit

[GitHub] spark issue #17768: [SPARK-20465][CORE] Throws a proper exception when any t...

2017-04-25 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/17768 @joshrosen, could you take a look and see if it makes sense? --- 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 issue #17768: [SPARK-20465][CORE] Throws a proper exception when any t...

2017-04-25 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/17768 This actually also happens when the directory exists but the user does not have the permission. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #17768: [SPARK-20465][CORE] Throws a proper exception when any t...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17768 **[Test build #76172 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76172/testReport)** for PR 17768 at commit

[GitHub] spark issue #17768: [SPARK-20465][CORE] Throws a proper exception when any t...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17768 **[Test build #76171 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76171/testReport)** for PR 17768 at commit

[GitHub] spark issue #17737: [SPARK-20442][PYTHON][DOCS] Fill up documentations for f...

2017-04-25 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/17737 @holdenk do you have bandwidth to review this or ok with me pushing this to master? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request #17130: [SPARK-19791] [ML] Add doc and example for fpgrow...

2017-04-25 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/17130#discussion_r113366111 --- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala --- @@ -268,12 +269,8 @@ class FPGrowthModel private[ml] ( val predictUDF

[GitHub] spark pull request #17728: [SPARK-20437][R] R wrappers for rollup and cube

2017-04-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/17728 --- 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 #17768: [SPARK-20465][CORE] Throws a proper exception whe...

2017-04-25 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/17768 [SPARK-20465][CORE] Throws a proper exception when any temp directory could not be got/created (rather than ArrayIndexOutOfBoundsException) ## What changes were proposed in this pull request?

[GitHub] spark issue #17728: [SPARK-20437][R] R wrappers for rollup and cube

2017-04-25 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/17728 merged to 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 this feature enabled and wishes

[GitHub] spark issue #17191: [SPARK-14471][SQL] Aliases in SELECT could be used in GR...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17191 **[Test build #76170 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76170/testReport)** for PR 17191 at commit

[GitHub] spark pull request #17191: [SPARK-14471][SQL] Aliases in SELECT could be use...

2017-04-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/17191#discussion_r113365310 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -136,6 +136,7 @@ class Analyzer(

[GitHub] spark issue #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17757 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76168/ Test PASSed. ---

[GitHub] spark issue #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17757 **[Test build #76168 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76168/testReport)** for PR 17757 at commit

[GitHub] spark issue #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17757 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

[GitHub] spark issue #17596: [SPARK-12837][CORE] Do not send the name of internal acc...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17596 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76157/ Test FAILed. ---

[GitHub] spark issue #17596: [SPARK-12837][CORE] Do not send the name of internal acc...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17596 **[Test build #76157 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76157/testReport)** for PR 17596 at commit

[GitHub] spark issue #17596: [SPARK-12837][CORE] Do not send the name of internal acc...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17596 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

[GitHub] spark pull request #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/17757#discussion_r113362917 --- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R --- @@ -284,22 +284,11 @@ test_that("spark.mlp", { c("1.0", "1.0",

[GitHub] spark pull request #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/17757#discussion_r113362748 --- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R --- @@ -284,22 +284,11 @@ test_that("spark.mlp", { c("1.0", "1.0",

[GitHub] spark issue #17733: [SPARK-20425][SQL] Support an extended display mode for ...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17733 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

[GitHub] spark issue #17733: [SPARK-20425][SQL] Support an extended display mode for ...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17733 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76166/ Test PASSed. ---

[GitHub] spark issue #17733: [SPARK-20425][SQL] Support an extended display mode for ...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17733 **[Test build #76166 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76166/testReport)** for PR 17733 at commit

[GitHub] spark pull request #17640: [SPARK-17608][SPARKR]:Long type has incorrect ser...

2017-04-25 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/17640#discussion_r113362108 --- Diff: R/pkg/inst/tests/testthat/test_Serde.R --- @@ -28,6 +28,10 @@ test_that("SerDe of primitive types", { expect_equal(x, 1)

[GitHub] spark issue #17503: [SPARK-3159][MLlib] Check for reducible DecisionTree

2017-04-25 Thread facaiy
Github user facaiy commented on the issue: https://github.com/apache/spark/pull/17503 @srowen I am not sure whether I understand your question clearly. RandomForest uses LearningNode to construct tree model when training, and convert them to Leaf or InternalNode at last. Hence, all

[GitHub] spark pull request #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/17757#discussion_r113361559 --- Diff: R/pkg/inst/tests/testthat/test_mllib_classification.R --- @@ -284,22 +284,11 @@ test_that("spark.mlp", { c("1.0", "1.0",

[GitHub] spark pull request #17191: [SPARK-14471][SQL] Aliases in SELECT could be use...

2017-04-25 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/17191#discussion_r113360559 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -136,6 +136,7 @@ class Analyzer(

[GitHub] spark pull request #17503: [SPARK-3159][MLlib] Check for reducible DecisionT...

2017-04-25 Thread facaiy
Github user facaiy commented on a diff in the pull request: https://github.com/apache/spark/pull/17503#discussion_r113360409 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/tree/configuration/Strategy.scala --- @@ -61,6 +61,8 @@ import

[GitHub] spark pull request #16618: [SPARK-14409][ML][WIP] Add RankingEvaluator

2017-04-25 Thread ebernhardson
Github user ebernhardson commented on a diff in the pull request: https://github.com/apache/spark/pull/16618#discussion_r113360277 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingMetrics.scala --- @@ -0,0 +1,202 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #16618: [SPARK-14409][ML][WIP] Add RankingEvaluator

2017-04-25 Thread ebernhardson
Github user ebernhardson commented on a diff in the pull request: https://github.com/apache/spark/pull/16618#discussion_r113358325 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingEvaluator.scala --- @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #16618: [SPARK-14409][ML][WIP] Add RankingEvaluator

2017-04-25 Thread ebernhardson
Github user ebernhardson commented on a diff in the pull request: https://github.com/apache/spark/pull/16618#discussion_r113355473 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/RankingMetrics.scala --- @@ -0,0 +1,202 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #17596: [SPARK-12837][CORE] Do not send the name of internal acc...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17596 **[Test build #76169 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76169/testReport)** for PR 17596 at commit

[GitHub] spark issue #17757: [Minor][ML] Fix some PySpark & SparkR flaky tests

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17757 **[Test build #76168 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76168/testReport)** for PR 17757 at commit

[GitHub] spark pull request #17693: [SPARK-16548][SQL] Inconsistent error handling in...

2017-04-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/17693 --- 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 issue #17693: [SPARK-16548][SQL] Inconsistent error handling in JSON p...

2017-04-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/17693 thanks, merging to master/2.2 --- 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 issue #17556: [SPARK-16957][MLlib] Use weighted midpoints for split va...

2017-04-25 Thread facaiy
Github user facaiy commented on the issue: https://github.com/apache/spark/pull/17556 fix failed case, please retest it. --- 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 issue #17760: [SPARK-20439] [SQL] [Backport-2.1] Fix Catalog API listT...

2017-04-25 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/17760 thanks, merging to 2.1 --- 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

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17758 **[Test build #76167 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76167/testReport)** for PR 17758 at commit

[GitHub] spark pull request #17640: [SPARK-17608][SPARKR]:Long type has incorrect ser...

2017-04-25 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/17640#discussion_r113358460 --- Diff: R/pkg/inst/tests/testthat/test_Serde.R --- @@ -28,6 +28,10 @@ test_that("SerDe of primitive types", { expect_equal(x, 1)

[GitHub] spark pull request #17640: [SPARK-17608][SPARKR]:Long type has incorrect ser...

2017-04-25 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/17640#discussion_r113358355 --- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R --- @@ -3043,6 +3043,23 @@ test_that("catalog APIs, currentDatabase, setCurrentDatabase,

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17758 **[Test build #76165 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76165/testReport)** for PR 17758 at commit

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17758 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76165/ Test FAILed. ---

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17758 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

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 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

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76163/ Test PASSed. ---

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76163 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76163/testReport)** for PR 17765 at commit

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 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

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76162/ Test PASSed. ---

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76162 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76162/testReport)** for PR 17765 at commit

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 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

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76160/ Test PASSed. ---

[GitHub] spark pull request #17744: [SPARK-20426] Lazy initialization of FileSegmentM...

2017-04-25 Thread jinxing64
Github user jinxing64 commented on a diff in the pull request: https://github.com/apache/spark/pull/17744#discussion_r113356306 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockHandler.java --- @@ -93,14 +92,25 @@ protected void

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76160 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76160/testReport)** for PR 17765 at commit

[GitHub] spark issue #17649: [SPARK-20380][SQL] Output table comment for DESC FORMATT...

2017-04-25 Thread wzhfy
Github user wzhfy commented on the issue: https://github.com/apache/spark/pull/17649 The changes look good to me if we don't care about the case sensitivity issue. --- 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 issue #17766: [SPARK-20421][core] Mark internal listeners as deprecate...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17766 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76154/ Test PASSed. ---

[GitHub] spark issue #17725: [SPARK-20435][CORE] More thorough redaction of sensitive...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17725 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

[GitHub] spark issue #17766: [SPARK-20421][core] Mark internal listeners as deprecate...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17766 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

[GitHub] spark issue #17725: [SPARK-20435][CORE] More thorough redaction of sensitive...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17725 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76156/ Test FAILed. ---

[GitHub] spark issue #17766: [SPARK-20421][core] Mark internal listeners as deprecate...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17766 **[Test build #76154 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76154/testReport)** for PR 17766 at commit

[GitHub] spark issue #17725: [SPARK-20435][CORE] More thorough redaction of sensitive...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17725 **[Test build #76156 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76156/testReport)** for PR 17725 at commit

[GitHub] spark issue #17767: Refactoring of the ALS code

2017-04-25 Thread hhbyyh
Github user hhbyyh commented on the issue: https://github.com/apache/spark/pull/17767 Preparing a PR like this takes a lot of efforts. Please try to follow the guidelines in http://spark.apache.org/contributing.html. (create a jira and rename the title). Like you said, I

[GitHub] spark issue #17738: [SPARK-20422][Spark Core] Worker registration retries sh...

2017-04-25 Thread unsleepy22
Github user unsleepy22 commented on the issue: https://github.com/apache/spark/pull/17738 Could someone 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 not have this feature enabled and

[GitHub] spark pull request #17733: [SPARK-20425][SQL] Support an extended display mo...

2017-04-25 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/17733#discussion_r113351508 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -663,8 +695,54 @@ class Dataset[T] private[sql]( * @group action

[GitHub] spark issue #17733: [SPARK-20425][SQL] Support an extended display mode for ...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17733 **[Test build #76166 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76166/testReport)** for PR 17733 at commit

[GitHub] spark pull request #17191: [SPARK-14471][SQL] Aliases in SELECT could be use...

2017-04-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/17191#discussion_r113351326 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -136,6 +136,7 @@ class Analyzer(

[GitHub] spark issue #17760: [SPARK-20439] [SQL] [Backport-2.1] Fix Catalog API listT...

2017-04-25 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17760 cc @cloud-fan @sameeragarwal --- 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 issue #17693: [SPARK-16548][SQL] Inconsistent error handling in JSON p...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17693 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76153/ Test PASSed. ---

[GitHub] spark issue #17693: [SPARK-16548][SQL] Inconsistent error handling in JSON p...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17693 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

[GitHub] spark issue #17693: [SPARK-16548][SQL] Inconsistent error handling in JSON p...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17693 **[Test build #76153 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76153/testReport)** for PR 17693 at commit

[GitHub] spark pull request #17737: [SPARK-20442][PYTHON][DOCS] Fill up documentation...

2017-04-25 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/17737#discussion_r113350427 --- Diff: python/pyspark/sql/column.py --- @@ -251,15 +285,16 @@ def __iter__(self): # string methods _rlike_doc = """ -

[GitHub] spark issue #17077: [SPARK-16931][PYTHON][SQL] Add Python wrapper for bucket...

2017-04-25 Thread zero323
Github user zero323 commented on the issue: https://github.com/apache/spark/pull/17077 🙁 --- 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

[GitHub] spark pull request #16781: [SPARK-12297][SQL] Hive compatibility for Parquet...

2017-04-25 Thread ueshin
Github user ueshin commented on a diff in the pull request: https://github.com/apache/spark/pull/16781#discussion_r113346208 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/ParquetHiveCompatibilitySuite.scala --- @@ -397,13 +392,38 @@ class

[GitHub] spark pull request #16781: [SPARK-12297][SQL] Hive compatibility for Parquet...

2017-04-25 Thread ueshin
Github user ueshin commented on a diff in the pull request: https://github.com/apache/spark/pull/16781#discussion_r113345272 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/ParquetHiveCompatibilitySuite.scala --- @@ -17,14 +17,25 @@ package

[GitHub] spark issue #17758: [SPARK-20460][SQL] Make it more consistent to handle col...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17758 **[Test build #76165 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76165/testReport)** for PR 17758 at commit

[GitHub] spark issue #17767: Refactoring of the ALS code

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17767 Can one of the admins verify this patch? --- 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 #17767: Als refactor

2017-04-25 Thread danielyli
GitHub user danielyli opened a pull request: https://github.com/apache/spark/pull/17767 Als refactor ## What changes were proposed in this pull request? This is a non-feature-changing refactoring of the ALS code (specifically, the `org.apache.spark.ml.recommendation`

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17605 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76164/ Test PASSed. ---

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17605 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

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17605 **[Test build #76164 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76164/testReport)** for PR 17605 at commit

[GitHub] spark issue #17077: [SPARK-16931][PYTHON][SQL] Add Python wrapper for bucket...

2017-04-25 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/17077 I think we should because branch-2.2 is cut out. --- 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 #17733: [SPARK-20425][SQL] Support an extended display mo...

2017-04-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/17733#discussion_r113347799 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -663,8 +695,54 @@ class Dataset[T] private[sql]( * @group action

[GitHub] spark pull request #17733: [SPARK-20425][SQL] Support an extended display mo...

2017-04-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/17733#discussion_r113347676 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -663,8 +695,54 @@ class Dataset[T] private[sql]( * @group action

[GitHub] spark pull request #17733: [SPARK-20425][SQL] Support an extended display mo...

2017-04-25 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/17733#discussion_r113347666 --- Diff: R/pkg/R/DataFrame.R --- @@ -194,6 +194,8 @@ setMethod("isLocal", #' 20 characters will be truncated. However, if set greater

[GitHub] spark issue #17077: [SPARK-16931][PYTHON][SQL] Add Python wrapper for bucket...

2017-04-25 Thread zero323
Github user zero323 commented on the issue: https://github.com/apache/spark/pull/17077 @holdenk, @HyukjinKwon Do we retarget this to 2.3? --- 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 issue #17728: [SPARK-20437][R] R wrappers for rollup and cube

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17728 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

[GitHub] spark issue #17728: [SPARK-20437][R] R wrappers for rollup and cube

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17728 **[Test build #76158 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76158/testReport)** for PR 17728 at commit

[GitHub] spark issue #17728: [SPARK-20437][R] R wrappers for rollup and cube

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17728 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76158/ Test PASSed. ---

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17605 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

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17605 **[Test build #76161 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76161/testReport)** for PR 17605 at commit

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17605 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76161/ Test PASSed. ---

[GitHub] spark issue #17605: [SPARK-20290][MINOR][PYTHON][SQL] Add PySpark wrapper fo...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17605 **[Test build #76164 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76164/testReport)** for PR 17605 at commit

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and description for st...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76163 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76163/testReport)** for PR 17765 at commit

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and an informative des...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76152/ Test PASSed. ---

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and an informative des...

2017-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17765 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

[GitHub] spark issue #17765: [SPARK-20464][SS] Add a job group and an informative des...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76152 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76152/testReport)** for PR 17765 at commit

[GitHub] spark issue #17761: [SPARK-20461][Core][SS]Use UninterruptibleThread for Exe...

2017-04-25 Thread mridulm
Github user mridulm commented on the issue: https://github.com/apache/spark/pull/17761 @zsxwing Got it, thanks for clarifying. --- 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 issue #17765: [SPARK-20464][SS] Add a job group and an informative des...

2017-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17765 **[Test build #76162 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76162/testReport)** for PR 17765 at commit

  1   2   3   4   >