[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193585815 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193585535 **[Test build #52628 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52628/consoleFull)** for PR 11569 at commit

[GitHub] spark pull request: [SPARK-13600] [MLlib] [WIP] Incorrect number o...

2016-03-07 Thread oliverpierson
Github user oliverpierson commented on the pull request: https://github.com/apache/spark/pull/11553#issuecomment-193584221 Putting this up for review now. Tests are passing on my machine. Using `approxQuantile` in DataFrame stats reduces amount of code required by a good bit.

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11565#issuecomment-193583045 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-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11565#issuecomment-193583046 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11565#issuecomment-193582844 **[Test build #52620 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52620/consoleFull)** for PR 11565 at commit

[GitHub] spark pull request: [SPARK-13667][SQL] Support for specifying cust...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193582705 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-13667][SQL] Support for specifying cust...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193582706 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13667][SQL] Support for specifying cust...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193582028 **[Test build #52621 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52621/consoleFull)** for PR 11550 at commit

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193580343 **[Test build #52628 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52628/consoleFull)** for PR 11569 at commit

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/11565#discussion_r55311096 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -345,8 +343,6 @@ object ColumnPruning extends

[GitHub] spark pull request: [SPARK-13249][SQL] Add Filter checking nullabi...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11235#issuecomment-193579525 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-13249][SQL] Add Filter checking nullabi...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11235#issuecomment-193579527 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13249][SQL] Add Filter checking nullabi...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11235#issuecomment-193578999 **[Test build #52617 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52617/consoleFull)** for PR 11235 at commit

[GitHub] spark pull request: [HOT-FIX][BUILD] Use the new location of `chec...

2016-03-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11567#issuecomment-193578502 `PySpark` failure is irrelevant for this PR, but I rebased this PR to the master because this is still a problem. --- If your project is set up for it, you can

[GitHub] spark pull request: [HOT-FIX][BUILD] Use the new location of `chec...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11567#issuecomment-193578669 **[Test build #52627 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52627/consoleFull)** for PR 11567 at commit

[GitHub] spark pull request: [SPARK-13667][SQL] Support for specifying cust...

2016-03-07 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193576069 @falaki Just let to know, I changed the name `CSVInferSchema` to `InferSchema` mainly for consistent names for CSV and JSON data source but maybe they might have

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193573893 **[Test build #52626 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52626/consoleFull)** for PR 11555 at commit

[GitHub] spark pull request: [SPARK-9325][SPARK-R] collect() head() and sho...

2016-03-07 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11336#issuecomment-193573753 I personally find it confusing having to reason about when we can "head"/"collect"/"show" and when we cannot, and that's why the Scala/Python version of the API didn't

[GitHub] spark pull request: [SPARK-12719][SQL] SQL generation support for ...

2016-03-07 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11563#issuecomment-193572743 "as we wanted to generate SQLs which is closer to the original SQL" Why is this a goal? I worry about the fragility of this two cases, if we really only need one

[GitHub] spark pull request: [SPARK-13249][SQL] Add Filter checking nullabi...

2016-03-07 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/11235#discussion_r55309414 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -144,6 +146,56 @@ object EliminateSerialization extends

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193566395 It seems better to keep SparkR as a base package providing core functionalities, while visualization features can be implemented in other packages based on SparkR.

[GitHub] spark pull request: [SPARK-12719][SQL] SQL generation support for ...

2016-03-07 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/11563#discussion_r55309116 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/LogicalPlanToSQLSuite.scala --- @@ -445,4 +461,86 @@ class LogicalPlanToSQLSuite extends

[GitHub] spark pull request: [SPARK-13738][SQL] Cleanup Data Source resolut...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11572#issuecomment-193565461 **[Test build #52625 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52625/consoleFull)** for PR 11572 at commit

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193562698 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-10387][ML] Add code gen for gbt

2016-03-07 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/9524#discussion_r55308566 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/codeGenerator.scala --- @@ -0,0 +1,156 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193562700 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread mwws
Github user mwws closed the pull request at: https://github.com/apache/spark/pull/11571 --- 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-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193562358 **[Test build #52616 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52616/consoleFull)** for PR 11555 at commit

[GitHub] spark pull request: [SPARK-13738][SQL] Cleanup Data Source resolut...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11572#issuecomment-193562059 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13738][SQL] Cleanup Data Source resolut...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11572#issuecomment-193562058 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-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread mwws
Github user mwws commented on the pull request: https://github.com/apache/spark/pull/11571#issuecomment-193562023 OK, thanks for the explanation, I will close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-13738][SQL] Cleanup Data Source resolut...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11572#issuecomment-193562054 **[Test build #52624 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52624/consoleFull)** for PR 11572 at commit

[GitHub] spark pull request: [SPARK-13738][SQL] Cleanup Data Source resolut...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11572#issuecomment-193561759 **[Test build #52624 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52624/consoleFull)** for PR 11572 at commit

[GitHub] spark pull request: [SPARK-9325][SPARK-R] collect() head() and sho...

2016-03-07 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/11336#issuecomment-193561063 A column can be applied to different dataframes. For example, if both df1 and df2 have a column named "col",then col <- column("col") collect(select(df1,

[GitHub] spark pull request: [SPARK-13695] Don't cache MEMORY_AND_DISK bloc...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11533#issuecomment-193560625 **[Test build #2615 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2615/consoleFull)** for PR 11533 at commit

[GitHub] spark pull request: [SPARK-13664][SQL] Cleanup Data Source resolut...

2016-03-07 Thread marmbrus
GitHub user marmbrus opened a pull request: https://github.com/apache/spark/pull/11572 [SPARK-13664][SQL] Cleanup Data Source resolution Follow-up to #11509, that simply refactors the interface that we use when resolving a pluggable `DataSource`. - Multiple functions share the

[GitHub] spark pull request: [SPARK-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11571#issuecomment-193560334 But this can only be added to 2.0 (we won't be able to change an existing release). If users already need to change the constructor in order to use it, why don't they

[GitHub] spark pull request: [SPARK-13432][SQL] add the source file name an...

2016-03-07 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/11301#discussion_r55307944 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExpressionSet.scala --- @@ -50,7 +50,7 @@ object ExpressionSet { class

[GitHub] spark pull request: [SPARK-13527] [SQL] Prune Filters based on Con...

2016-03-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/11406#discussion_r55307963 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -769,6 +770,28 @@ object CombineFilters extends

[GitHub] spark pull request: [SPARK-13527] [SQL] Prune Filters based on Con...

2016-03-07 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/11406#discussion_r55307448 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -769,6 +770,28 @@ object CombineFilters extends

[GitHub] spark pull request: [HOT-FIX][BUILD] Use the new location of `chec...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11567#issuecomment-19389 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: [HOT-FIX][BUILD] Use the new location of `chec...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11567#issuecomment-19395 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread mwws
Github user mwws commented on the pull request: https://github.com/apache/spark/pull/11571#issuecomment-19338 @rxin HiveContext is heavily used by many users now, and many of them still coupled with old spark version. As this change would be trivial but not constructive, I think

[GitHub] spark pull request: [HOT-FIX][BUILD] Use the new location of `chec...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11567#issuecomment-193555142 **[Test build #52614 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52614/consoleFull)** for PR 11567 at commit

[GitHub] spark pull request: [SPARK-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11571#issuecomment-193555075 Thanks for the pull request. So we are actually going to deprecate HiveContext because it has been one of the most confusing contexts in Spark. See more in

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11557#issuecomment-193554969 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11557#issuecomment-193554967 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-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11557#issuecomment-193554765 **[Test build #52613 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52613/consoleFull)** for PR 11557 at commit

[GitHub] spark pull request: [SPARK-13689] [SQL] Move helper things in Cata...

2016-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11529 --- 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-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11571#issuecomment-193553927 **[Test build #52623 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52623/consoleFull)** for PR 11571 at commit

[GitHub] spark pull request: [SPARK-13689] [SQL] Move helper things in Cata...

2016-03-07 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11529#issuecomment-193553008 Merging 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

[GitHub] spark pull request: [SPARK-13737][SQL][wip]Add getOrCreate method ...

2016-03-07 Thread mwws
GitHub user mwws opened a pull request: https://github.com/apache/spark/pull/11571 [SPARK-13737][SQL][wip]Add getOrCreate method for HiveContext There is a "getOrCreate" method in SQLContext, which is useful to recoverable streaming application with SQL operation.

[GitHub] spark pull request: [SPARK-529] [sql] Modify SQLConf to use new co...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11570#issuecomment-193552569 **[Test build #52622 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52622/consoleFull)** for PR 11570 at commit

[GitHub] spark pull request: [SPARK-529] [sql] Modify SQLConf to use new co...

2016-03-07 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/11570 [SPARK-529] [sql] Modify SQLConf to use new config API from core. Because SQL keeps track of all known configs, some customization was needed in SQLConf to allow that, since the core API does

[GitHub] spark pull request: [SPARK-13692][CORE][SQL] Fix trivial Coverity/...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11530#issuecomment-193552260 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-13692][CORE][SQL] Fix trivial Coverity/...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11530#issuecomment-193552263 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13692][CORE][SQL] Fix trivial Coverity/...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11530#issuecomment-193552052 **[Test build #52611 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52611/consoleFull)** for PR 11530 at commit

[GitHub] spark pull request: [SPARK-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193551318 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193551316 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-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193551056 **[Test build #52609 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52609/consoleFull)** for PR 11566 at commit

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193550601 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193550529 **[Test build #52618 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52618/consoleFull)** for PR 11569 at commit

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193550598 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-13667][SQL] Support for specifying cust...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193550315 **[Test build #52621 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52621/consoleFull)** for PR 11550 at commit

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11565#issuecomment-193550327 **[Test build #52620 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52620/consoleFull)** for PR 11565 at commit

[GitHub] spark pull request: [SPARK-13667][SQL] Support for specifying cust...

2016-03-07 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193549873 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-13667][SQL] Support for specifying cust...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11550#issuecomment-193549215 **[Test build #52619 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52619/consoleFull)** for PR 11550 at commit

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/11565#discussion_r55304511 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -345,8 +343,6 @@ object ColumnPruning extends

[GitHub] spark pull request: [SPARK-13404] [SQL] Create variables for input...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11274#issuecomment-193548355 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13404] [SQL] Create variables for input...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11274#issuecomment-193548354 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-13404] [SQL] Create variables for input...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11274#issuecomment-193547869 **[Test build #52612 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52612/consoleFull)** for PR 11274 at commit

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193545016 **[Test build #52618 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52618/consoleFull)** for PR 11569 at commit

[GitHub] spark pull request: [SPARK-13577] [yarn] Allow Spark jar to be mul...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11500#issuecomment-193543523 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13577] [yarn] Allow Spark jar to be mul...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11500#issuecomment-193543521 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-13577] [yarn] Allow Spark jar to be mul...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11500#issuecomment-193542877 **[Test build #52607 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52607/consoleFull)** for PR 11500 at commit

[GitHub] spark pull request: [SPARK-12719][SQL] SQL generation support for ...

2016-03-07 Thread dilipbiswal
Github user dilipbiswal commented on the pull request: https://github.com/apache/spark/pull/11563#issuecomment-193542558 @rxin. Hi Reynold, We have two cases to handle. ```SQL SELECT explode(array(1,2,3)) FROM src SELECT gentab2.* FROM t1 LATERAL VIEW

[GitHub] spark pull request: [SPARK-11108][ML] OneHotEncoder should support...

2016-03-07 Thread frreiss
Github user frreiss commented on the pull request: https://github.com/apache/spark/pull/9777#issuecomment-193542138 LGTM aside from that one typo --- 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-11108][ML] OneHotEncoder should support...

2016-03-07 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/9777#discussion_r55302705 --- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/OneHotEncoder.scala --- @@ -132,8 +133,10 @@ class OneHotEncoder(override val uid: String) extends

[GitHub] spark pull request: [SPARK-11535][ML] handling empty string in Str...

2016-03-07 Thread thunterdb
Github user thunterdb commented on the pull request: https://github.com/apache/spark/pull/9522#issuecomment-193542005 @pravingadakh sorry for the delay. Would you mind resolving the conflicts? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13249][SQL] Add Filter checking nullabi...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11235#issuecomment-193541806 **[Test build #52617 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52617/consoleFull)** for PR 11235 at commit

[GitHub] spark pull request: [SPARK-13549] [SQL] Refactor the Optimizer Rul...

2016-03-07 Thread frreiss
Github user frreiss commented on the pull request: https://github.com/apache/spark/pull/11427#issuecomment-193540108 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-13117] [Web UI] WebUI should use the lo...

2016-03-07 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/11490#issuecomment-193539323 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-13734][SPARKR] Added histogram function

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193538403 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-13734][SPARKR] Added histogram function

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193538292 **[Test build #52615 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52615/consoleFull)** for PR 11569 at commit

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-193538409 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193537547 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

[GitHub] spark pull request: [SPARK-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11565#discussion_r55301619 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -345,8 +343,6 @@ object ColumnPruning extends

[GitHub] spark pull request: [SPARK-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11566#discussion_r55301512 --- Diff: core/src/main/scala/org/apache/spark/deploy/client/AppClient.scala --- @@ -125,16 +125,14 @@ private[spark] class AppClient(

[GitHub] spark pull request: [SPARK-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193534923 **[Test build #52616 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52616/consoleFull)** for PR 11555 at commit

[GitHub] spark pull request: [SPARK-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193534718 > So what happens now if the scheduled Runnable throws an exception? Just go to `Thread.getDefaultUncaughtExceptionHandler()`. --- If your project is set up

[GitHub] spark pull request: [SPARK-13675][UI] Fix wrong historyserver url ...

2016-03-07 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/11518#issuecomment-193534057 Yes, I also tested with multiple attempts. --- 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-13404] [SQL] Create variables for input...

2016-03-07 Thread nongli
Github user nongli commented on the pull request: https://github.com/apache/spark/pull/11274#issuecomment-193534012 sounds good. --- 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-12718][SPARK-13720][SQL] SQL generation...

2016-03-07 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11555#issuecomment-193533951 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-13732] [SQL] Remove projectList from Wi...

2016-03-07 Thread frreiss
Github user frreiss commented on the pull request: https://github.com/apache/spark/pull/11565#issuecomment-193533582 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-13711][Core]Don't call SparkUncaughtExc...

2016-03-07 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11566#issuecomment-193533661 So what happens now if the scheduled Runnable throws an exception? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-11011][SQL] Narrow type of UDT serializ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11379#issuecomment-193531378 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11011][SQL] Narrow type of UDT serializ...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11379#issuecomment-193531376 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-12469][CORE][WIP/RFC] Consistent accumu...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11105#issuecomment-193531213 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12469][CORE][WIP/RFC] Consistent accumu...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11105#issuecomment-193531212 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   >