[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175971832 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

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175971830 **[Test build #50249 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50249/consoleFull)** for PR 9893 at commit [`75cb0f6`](https://git

[GitHub] spark pull request: [SPARK-12989] [SQL] Delaying Alias Cleanup aft...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10963#issuecomment-175971750 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 projec

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175971770 **[Test build #50244 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50244/consoleFull)** for PR 10958 at commit [`6e4859d`](https://g

[GitHub] spark pull request: [SPARK-12989] [SQL] Delaying Alias Cleanup aft...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10963#issuecomment-175971754 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12989] [SQL] Delaying Alias Cleanup aft...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10963#issuecomment-175971589 **[Test build #50246 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50246/consoleFull)** for PR 10963 at commit [`468abca`](https://g

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175970101 **[Test build #50249 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50249/consoleFull)** for PR 9893 at commit [`75cb0f6`](https://gith

[GitHub] spark pull request: [SPARK-10873] Support column sort and search f...

2016-01-27 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/spark/pull/10648#issuecomment-175966503 Hi @tgravescs , finally fixed the paging stuff in RowsGrouping. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51077616 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51077516 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175955464 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

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175955469 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175955462 **[Test build #50247 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50247/consoleFull)** for PR 9893 at commit [`ab6d601`](https://git

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-175955168 **[Test build #50247 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50247/consoleFull)** for PR 9893 at commit [`ab6d601`](https://gith

[GitHub] spark pull request: [SPARK-13047][PYSPARK][ML] Pyspark Params.hasP...

2016-01-27 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/10962#discussion_r51076979 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -152,13 +152,17 @@ def isDefined(self, param): return self.isSet(param) or self.hasDefault(pa

[GitHub] spark pull request: [SPARK-12792][SPARKR] Refactor RRDD to support...

2016-01-27 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/10947#issuecomment-175946838 cc @davies. Thanks @sunrui for the PR. I'll review this later today --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-13047][PYSPARK][ML] Pyspark Params.hasP...

2016-01-27 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/10962#discussion_r51075753 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -152,13 +152,17 @@ def isDefined(self, param): return self.isSet(param) or self.hasDefaul

[GitHub] spark pull request: [SPARK-12989] [SQL] Delaying Alias Cleanup aft...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10963#issuecomment-175939401 **[Test build #50246 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50246/consoleFull)** for PR 10963 at commit [`468abca`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r51074963 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlLexer.g --- @@ -465,7 +467,7 @@ Identifier fragment QuotedI

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r51074937 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlLexer.g --- @@ -465,7 +467,7 @@ Identifier fragment QuotedI

[GitHub] spark pull request: [SPARK-12989] [SQL] Delaying Alias Cleanup aft...

2016-01-27 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/10963 [SPARK-12989] [SQL] Delaying Alias Cleanup after ExtractWindowExpressions In the rule `ExtractWindowExpressions`, we simply replace alias by the name. However, this will cause an issue exposed b

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10527#issuecomment-175932477 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10527#issuecomment-175932469 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

[GitHub] spark pull request: [SPARK-13020][SQL][test] fix random generator ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10930#issuecomment-175932166 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 projec

[GitHub] spark pull request: [SPARK-13020][SQL][test] fix random generator ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10930#issuecomment-175932171 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13020][SQL][test] fix random generator ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10930#issuecomment-175931946 **[Test build #50233 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50233/consoleFull)** for PR 10930 at commit [`e627f5b`](https://g

[GitHub] spark pull request: [SPARK-12656] [SQL] Implement Intersect with L...

2016-01-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10630#discussion_r51074011 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -388,57 +445,18 @@ class Analyzer( .m

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175930915 **[Test build #50244 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50244/consoleFull)** for PR 10958 at commit [`6e4859d`](https://gi

[GitHub] spark pull request: [SPARK-12968][SQL] Implement command to set cu...

2016-01-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10916#discussion_r51073555 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/ClientInterface.scala --- @@ -109,6 +109,9 @@ private[hive] trait ClientInterface {

[GitHub] spark pull request: [SPARK-12968][SQL] Implement command to set cu...

2016-01-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10916#discussion_r51073506 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Catalog.scala --- @@ -46,6 +46,10 @@ trait Catalog { def lookupRela

[GitHub] spark pull request: [SPARK-11955][SQL] Mark optional fields in mer...

2016-01-27 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/9940#issuecomment-175927505 ping @liancheng Please see if latest updates are proper for you. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] spark pull request: [SPARK-12968][SQL] Implement command to set cu...

2016-01-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10916#discussion_r51073364 --- Diff: sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/CliSuite.scala --- @@ -183,7 +183,7 @@ class CliSuite extends SparkFunS

[GitHub] spark pull request: SPARK-11565 Replace deprecated DigestUtils.sha...

2016-01-27 Thread gliptak
Github user gliptak commented on the pull request: https://github.com/apache/spark/pull/9532#issuecomment-175927249 Github is hiccuping ... --- 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 thi

[GitHub] spark pull request: [SPARK-12567][SQL] Add aes_{encrypt,decrypt} U...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10527#issuecomment-175927638 **[Test build #50243 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50243/consoleFull)** for PR 10527 at commit [`9476822`](https://gi

[GitHub] spark pull request: [SPARK-12995][GraphX] Remove deprecate APIs fr...

2016-01-27 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10918#issuecomment-175927636 @ankurdave @srowen ping --- 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 thi

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175926617 retest 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-12567][SQL] Add aes_{encrypt,decrypt} U...

2016-01-27 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/10527#issuecomment-175924450 retest 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-13047][PYSPARK][ML] Pyspark Params.hasP...

2016-01-27 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/10962#issuecomment-175921366 I think this makes sense since it is harmonizing the behaviour with the Scala API. Perhaps adding a note in the PyDoc about the new behaviour as well as a test might be

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread iyounus
Github user iyounus commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51071809 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Sinc

[GitHub] spark pull request: [SPARK-13047][PYSPARK][ML] Pyspark Params.hasP...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10962#issuecomment-175920740 **[Test build #50242 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50242/consoleFull)** for PR 10962 at commit [`d52b1de`](https://gi

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread iyounus
Github user iyounus commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51071489 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Sinc

[GitHub] spark pull request: [SPARK-13047][PYSPARK][ML] Pyspark Params.hasP...

2016-01-27 Thread sethah
GitHub user sethah opened a pull request: https://github.com/apache/spark/pull/10962 [SPARK-13047][PYSPARK][ML] Pyspark Params.hasParam should not throw an error Pyspark Params class has a method `hasParam(paramName)` which returns `True` if the class has a parameter by that name, b

[GitHub] spark pull request: [SPARK-12988][SQL] Can't drop columns that con...

2016-01-27 Thread dilipbiswal
Github user dilipbiswal commented on the pull request: https://github.com/apache/spark/pull/10943#issuecomment-175917517 @cloud-fan Thank you Wenchen. --- 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 n

[GitHub] spark pull request: [SPARK-13043][SQL] Implement remaining catalys...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10961#issuecomment-175917592 **[Test build #50241 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50241/consoleFull)** for PR 10961 at commit [`24ca13c`](https://gi

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51070506 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [SPARK-10810] [SPARK-10902] [SQL] Improve sess...

2016-01-27 Thread Neuw84
Github user Neuw84 commented on the pull request: https://github.com/apache/spark/pull/8909#issuecomment-175916186 @deenar, I saw it after many hours reading the code in the web docs . Although, I think that the HiveSparkContext should implement the same logic as the SparkContext whe

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51070301 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [SPARK-12963] Improve performance of stddev/va...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10960#issuecomment-175915582 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 projec

[GitHub] spark pull request: [SPARK-12963] Improve performance of stddev/va...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10960#issuecomment-175915586 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51070105 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51070090 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51069962 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -219,33 +219,43 @@ class LinearRegression @Since("1.3.0") (@Since

[GitHub] spark pull request: [SPARK-12963] Improve performance of stddev/va...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10960#issuecomment-175914531 **[Test build #50240 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50240/consoleFull)** for PR 10960 at commit [`61edd5e`](https://gi

[GitHub] spark pull request: [SPARK-13049] Add First/last with ignore nulls...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10957#issuecomment-175913747 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 projec

[GitHub] spark pull request: [SPARK-13049] Add First/last with ignore nulls...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10957#issuecomment-175913754 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13043][SQL] Implement remaining catalys...

2016-01-27 Thread nongli
GitHub user nongli opened a pull request: https://github.com/apache/spark/pull/10961 [SPARK-13043][SQL] Implement remaining catalyst types in ColumnarBatch. This includes: float, boolean, short, decimal and calendar interval. Decimal is mapped to long or byte array depending

[GitHub] spark pull request: [SPARK-13049] Add First/last with ignore nulls...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10957#issuecomment-175913287 **[Test build #50231 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50231/consoleFull)** for PR 10957 at commit [`defcc02`](https://g

[GitHub] spark pull request: [SPARK-12963] Improve performance of stddev/va...

2016-01-27 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10960#issuecomment-175911728 cc @mengxr --- 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 en

[GitHub] spark pull request: [SPARK-13029][ml] fix a logistic regression is...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10940#issuecomment-175911585 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 projec

[GitHub] spark pull request: [WIP][SPARK-12957][SQL] Initial support for co...

2016-01-27 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10844#discussion_r51068802 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala --- @@ -17,16 +17,31 @@ package org.apache.spark.sql.c

[GitHub] spark pull request: [SPARK-13029][ml] fix a logistic regression is...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10940#issuecomment-175911586 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13045][SQL] Remove ColumnVector.Struct ...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10952 --- 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 ena

[GitHub] spark pull request: [WIP][SPARK-12957][SQL] Initial support for co...

2016-01-27 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10844#discussion_r51068566 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala --- @@ -146,6 +172,26 @@ case class Union(children:

[GitHub] spark pull request: [SPARK-13029][ml] fix a logistic regression is...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10940#issuecomment-175911254 **[Test build #50227 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50227/consoleFull)** for PR 10940 at commit [`914cffc`](https://g

[GitHub] spark pull request: [SPARK-13045][SQL] Remove ColumnVector.Struct ...

2016-01-27 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10952#issuecomment-175911147 Merging this into master, 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 no

[GitHub] spark pull request: [SPARK-12963] Improve performance of stddev/va...

2016-01-27 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/10960 [SPARK-12963] Improve performance of stddev/variance As benchmarked and discussed here: https://github.com/apache/spark/pull/10786/files#r50038294, benefits from codegen, the declarative aggregate

[GitHub] spark pull request: [WIP][SPARK-12957][SQL] Initial support for co...

2016-01-27 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10844#discussion_r51068499 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala --- @@ -88,6 +88,12 @@ case class Generate(

[GitHub] spark pull request: [WIP][SPARK-12957][SQL] Initial support for co...

2016-01-27 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10844#discussion_r51068376 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala --- @@ -17,16 +17,31 @@ package org.apache.spark.sql.c

[GitHub] spark pull request: [SPARK-13045][SQL] Remove ColumnVector.Struct ...

2016-01-27 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10952#issuecomment-175909805 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 a

[GitHub] spark pull request: [SPARK-13045][SQL] Remove ColumnVector.Struct ...

2016-01-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10952#issuecomment-175909089 cc @davies --- 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 enabl

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175908753 **[Test build #50238 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50238/consoleFull)** for PR 10958 at commit [`6e4859d`](https://gi

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175908426 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 projec

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175908428 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13049] Add First/last with ignore nulls...

2016-01-27 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10957#issuecomment-175908190 A user is trying to get this working on 1.6 using the dataframe api. That doesn't work directly because functions.scala misses the functions implemented in this PR.

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175907876 **[Test build #50235 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50235/consoleFull)** for PR 10958 at commit [`822899d`](https://g

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175907935 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-10620] [SPARK-13054] Minor addendum to ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175907933 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 projec

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175907259 **[Test build #50236 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50236/consoleFull)** for PR 10608 at commit [`18c5223`](https://g

[GitHub] spark pull request: [SPARK-12926][SQL] SQLContext to disallow user...

2016-01-27 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/10849#issuecomment-175907368 Please update the title and description (these become the commit message when merging). --- If your project is set up for it, you can reply to this email and have you

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175907305 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175907303 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 projec

[GitHub] spark pull request: [SPARK-12988][SQL] Can't drop columns that con...

2016-01-27 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10943#issuecomment-175906396 `select` is the API that supposed to take `column path`, but something like `withColumn`, `drop`, etc. is supposed to take `column name`. So what I suggest is: change

[GitHub] spark pull request: SPARK-13052 waitingApps metric doesn't show th...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10959#issuecomment-175904859 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 p

[GitHub] spark pull request: [SPARK-13049] Add First/last with ignore nulls...

2016-01-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10957#issuecomment-175904468 Why might this be a bug fix? --- 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-10620] Minor addendum to #10835

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175904183 **[Test build #50235 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50235/consoleFull)** for PR 10958 at commit [`822899d`](https://gi

[GitHub] spark pull request: SPARK-13052 waitingApps metric doesn't show th...

2016-01-27 Thread RaafatAkkad
GitHub user RaafatAkkad opened a pull request: https://github.com/apache/spark/pull/10959 SPARK-13052 waitingApps metric doesn't show the number of apps currently in the WAITING state You can merge this pull request into a Git repository by running: $ git pull https://github.

[GitHub] spark pull request: [HOTFIX] Fix Scala 2.11 compilation

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10955#issuecomment-175903354 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 projec

[GitHub] spark pull request: [HOTFIX] Fix Scala 2.11 compilation

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10955#issuecomment-175903359 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175903316 **[Test build #50236 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50236/consoleFull)** for PR 10608 at commit [`18c5223`](https://gi

[GitHub] spark pull request: [HOTFIX] Fix Scala 2.11 compilation

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10955#issuecomment-175903131 **[Test build #50223 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50223/consoleFull)** for PR 10955 at commit [`5b449cc`](https://g

[GitHub] spark pull request: [SPARK-10620] Minor addendum to #10835

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175903012 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 projec

[GitHub] spark pull request: [SPARK-10620] Minor addendum to #10835

2016-01-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175902964 **[Test build #50234 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50234/consoleFull)** for PR 10958 at commit [`5404254`](https://g

[GitHub] spark pull request: [SPARK-10620] Minor addendum to #10835

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175903014 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51064724 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -558,6 +575,47 @@ class LinearRegressionSuite }

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10702#discussion_r51064615 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -558,6 +575,47 @@ class LinearRegressionSuite }

[GitHub] spark pull request: [Spark-12732][ML] bug fix in linear regression...

2016-01-27 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/10702#issuecomment-175901752 @iyounus `standardizeLabel = false/ture` with non-zero `regParam`, let's throw the exception. I explained the mismatch against the analytic normal equation in the other

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175901624 @srowen, yep, this is pretty much just the result of running the version changing and dep update scripts, followed by a grep through the documentation and build to in

[GitHub] spark pull request: [SPARK-10620] Minor addendum to #10835

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175901408 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 projec

[GitHub] spark pull request: [SPARK-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-27 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10608#issuecomment-175901457 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-10620] Minor addendum to #10835

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10958#issuecomment-175901412 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

<    1   2   3   4   5   6   7   >