[GitHub] spark pull request #14682: [SPARK-17104][SQL] LogicalRelation.newInstance sh...

2016-08-19 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/14682#discussion_r75483271 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala --- @@ -79,11 +79,18 @@ case class LogicalRelation(

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

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

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14717 **[Test build #64069 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64069/consoleFull)** for PR 14717 at commit [`06c2147`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14717 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 e

[GitHub] spark pull request #14712: [SPARK-17072] [SQL] support table-level statistic...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r75481689 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -33,7 +34,7 @@ import org.apache.spark.sql.cata

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14717 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 enabled

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14717 **[Test build #64068 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64068/consoleFull)** for PR 14717 at commit [`396ba1d`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

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

[GitHub] spark issue #14717: [SPARK-17090][ML]Make tree aggregation level in linear/l...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14717 **[Test build #64068 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64068/consoleFull)** for PR 14717 at commit [`396ba1d`](https://github.com/apache/spark/commit/3

[GitHub] spark pull request #14717: [SPARK-17090][ML]Make tree aggregation level in l...

2016-08-19 Thread hqzizania
GitHub user hqzizania opened a pull request: https://github.com/apache/spark/pull/14717 [SPARK-17090][ML]Make tree aggregation level in linear/logistic regression configurable ## What changes were proposed in this pull request? Linear/logistic regression use treeAggregate w

[GitHub] spark pull request #14712: [SPARK-17072] [SQL] support table-level statistic...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r75480152 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -88,14 +90,30 @@ case class AnalyzeTableCommand(

[GitHub] spark pull request #14712: [SPARK-17072] [SQL] support table-level statistic...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r75479677 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -99,9 +99,7 @@ class SparkSqlAstBuilder(conf: SQLConf) extends

[GitHub] spark issue #14709: [SPARK-17150][SQL] Support SQL generation for inline tab...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14709 **[Test build #64067 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64067/consoleFull)** for PR 14709 at commit [`442918f`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #14709: [SPARK-17150][SQL] Support SQL generation for inline tab...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14709 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 feature enabled and wishes so,

[GitHub] spark issue #14124: [SPARK-16472][SQL] Inconsistent nullability in schema af...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14124 **[Test build #64066 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64066/consoleFull)** for PR 14124 at commit [`079aae2`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14713 thanks, merging to master/2.0! --- 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 an

[GitHub] spark pull request #14713: [SPARK-16994][SQL] Whitelist operators for predic...

2016-08-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14713 --- 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 issue #14707: [SPARK-17144] [SQL] Removal of useless CreateHiveTableAs...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14707 LGTM, but let's wait for https://github.com/apache/spark/pull/14155 to avoid conflict, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] spark pull request #14697: [SPARK-17124][SQL] RelationalGroupedDataset.agg s...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14697#discussion_r75477964 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala --- @@ -143,7 +143,9 @@ class RelationalGroupedDataset protected[sql]

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14279 **[Test build #64065 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64065/consoleFull)** for PR 14279 at commit [`405e723`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #14660: [SPARK-17071][SQL] Add an option to support for reading ...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14660 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 e

[GitHub] spark issue #14660: [SPARK-17071][SQL] Add an option to support for reading ...

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

[GitHub] spark issue #14660: [SPARK-17071][SQL] Add an option to support for reading ...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14660 **[Test build #64063 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64063/consoleFull)** for PR 14660 at commit [`1297a91`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14524 **[Test build #64064 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64064/consoleFull)** for PR 14524 at commit [`26fa2e0`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14524 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 e

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

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

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

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

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14279 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 e

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14279 **[Test build #64061 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64061/consoleFull)** for PR 14279 at commit [`7e61674`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14038: [SPARK-16317][SQL] Add a new interface to filter files i...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14038 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 e

[GitHub] spark issue #14038: [SPARK-16317][SQL] Add a new interface to filter files i...

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

[GitHub] spark issue #14038: [SPARK-16317][SQL] Add a new interface to filter files i...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14038 **[Test build #64062 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64062/consoleFull)** for PR 14038 at commit [`60f05ad`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14524 **[Test build #64064 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64064/consoleFull)** for PR 14524 at commit [`26fa2e0`](https://github.com/apache/spark/commit/2

[GitHub] spark pull request #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking ...

2016-08-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14555 --- 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 issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14555 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 so, or i

[GitHub] spark issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14555 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 e

[GitHub] spark issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

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

[GitHub] spark issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14555 **[Test build #64059 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64059/consoleFull)** for PR 14555 at commit [`b291423`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14524 **[Test build #64060 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64060/consoleFull)** for PR 14524 at commit [`ceb5050`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14524 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 e

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

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

[GitHub] spark issue #14678: [MINOR][SQL] Add missing functions for some options in S...

2016-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14678 (@rxin cc you in case you just missed this) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] spark issue #14660: [SPARK-17071][SQL] Add an option to support for reading ...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14660 **[Test build #64063 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64063/consoleFull)** for PR 14660 at commit [`1297a91`](https://github.com/apache/spark/commit/1

[GitHub] spark issue #14155: [SPARK-16498][SQL] move hive hack for data source table ...

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

[GitHub] spark issue #14155: [SPARK-16498][SQL] move hive hack for data source table ...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14155 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 e

[GitHub] spark issue #14155: [SPARK-16498][SQL] move hive hack for data source table ...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14155 **[Test build #64058 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64058/consoleFull)** for PR 14155 at commit [`6ca8909`](https://github.com/apache/spark/commit/

[GitHub] spark pull request #14660: [SPARK-17071][SQL] Add an option to support for r...

2016-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/14660#discussion_r75464633 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala --- @@ -243,6 +243,14 @@ object SQLConf { .booleanConf

[GitHub] spark issue #14038: [SPARK-16317][SQL] Add a new interface to filter files i...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14038 **[Test build #64062 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64062/consoleFull)** for PR 14038 at commit [`60f05ad`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14279 **[Test build #64061 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64061/consoleFull)** for PR 14279 at commit [`7e61674`](https://github.com/apache/spark/commit/7

[GitHub] spark issue #12790: [SPARK-15018][PYSPARK][ML] Fixed bug causing error if Py...

2016-08-19 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/12790 @MLnick Thanks for clarification. Then I think current fix is OK, since Scala is strongly typed, we can regard ```Array()``` or ```Array.empty``` as wrong input and throw exception. It's not nece

[GitHub] spark issue #14382: [SPARK-16673] [Web UI] New Executor Page removed conditi...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14382 I see, I tend to trust your work here. The result looks OK. Looks like this only needs to go to master --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #12790: [SPARK-15018][PYSPARK][ML] Fixed bug causing error if Py...

2016-08-19 Thread MLnick
Github user MLnick commented on the issue: https://github.com/apache/spark/pull/12790 Actually the behavior in Scala is consistent - passing in `Array()` or `Array.empty()` you are passing in `Array[Nothing]` so that is why there is a `ClassCastException`. Passing in an empty

[GitHub] spark issue #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14524 **[Test build #64060 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64060/consoleFull)** for PR 14524 at commit [`ceb5050`](https://github.com/apache/spark/commit/c

[GitHub] spark issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14555 **[Test build #64059 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64059/consoleFull)** for PR 14555 at commit [`b291423`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #14663: [SPARK-17001] [ML] Enable standardScaler to standardize ...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14663 Another one where I'd welcome comments from ... @holdenk @MLnick @davies et al --- 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 #14524: [SPARK-16832] [ML] [WIP] CrossValidator and TrainValidat...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14524 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 have this feature enabled and wishe

[GitHub] spark issue #14555: [SPARK-16965][MLLIB][PYSPARK] Fix bound checking for Spa...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14555 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 have this feature enabled and wishe

[GitHub] spark pull request #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN...

2016-08-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14716 --- 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 issue #14711: [SPARK-16822] [DOC] [Support latex in scaladoc with Math...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14711 You can just merge the work over to the other PR. I think we want to fix latex that's rendering as code accidentally, and latex whose closing braces are mangled by scaladoc. --- If your project is

[GitHub] spark issue #14711: [SPARK-16822] [DOC] [Support latex in scaladoc with Math...

2016-08-19 Thread jagadeesanas2
Github user jagadeesanas2 commented on the issue: https://github.com/apache/spark/pull/14711 ohh i got it. Can i 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 your project does not have this feature

[GitHub] spark issue #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN value.

2016-08-19 Thread yanboliang
Github user yanboliang commented on the issue: https://github.com/apache/spark/pull/14716 Merged into master. Thanks for review. --- 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 e

[GitHub] spark pull request #14181: [SPARK-15382][SQL] Fix a rule to push down projec...

2016-08-19 Thread maropu
Github user maropu closed the pull request at: https://github.com/apache/spark/pull/14181 --- 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 issue #14711: [SPARK-16822] [DOC] [Support latex in scaladoc with Math...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14711 I'm referring to your PR https://github.com/apache/spark/pull/14688 I don't see that he said he's working on it, just that he encountered it while working on something else. I'd rather fix the mathj

[GitHub] spark issue #14181: [SPARK-15382][SQL] Fix a rule to push down projects bene...

2016-08-19 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/14181 @viirya Ah, I noticed this issue has already been fixed in your pr #14327(SPARK-16686). So, I'll close this. Thanks! --- If your project is set up for it, you can reply to this email and have yo

[GitHub] spark issue #14666: [SPARK-16578][SparkR] Enable SparkR to connect to a remo...

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

[GitHub] spark issue #14666: [SPARK-16578][SparkR] Enable SparkR to connect to a remo...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14666 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 enabled

[GitHub] spark issue #14666: [SPARK-16578][SparkR] Enable SparkR to connect to a remo...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14666 **[Test build #64055 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64055/consoleFull)** for PR 14666 at commit [`54fe8a9`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14711: [SPARK-16822] [DOC] [Support latex in scaladoc with Math...

2016-08-19 Thread jagadeesanas2
Github user jagadeesanas2 commented on the issue: https://github.com/apache/spark/pull/14711 @srowen , Actually this fix (mathjax issues) for this JIRA only. As @sethah mentioned in his comment, he is working on `SPARK-17095`, he will send PR soon. --- If your project is set up for

[GitHub] spark issue #14181: [SPARK-15382][SQL] Fix a rule to push down projects bene...

2016-08-19 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/14181 @viirya I noticed this issues was resolved in SPARK-16686(#14327). So, I'll close this. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark issue #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN value.

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14716 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 and wishes so, or if the featur

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14279 Oh, wait, I think I can handle this. Please ignore the comments I just left. Sorry for noise. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark issue #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN value.

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

[GitHub] spark issue #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN value.

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14716 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 e

[GitHub] spark issue #14716: [SPARK-17141] [ML] MinMaxScaler should remain NaN value.

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14716 **[Test build #64057 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64057/consoleFull)** for PR 14716 at commit [`19cb3ad`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14713 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 e

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

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

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14713 **[Test build #64053 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64053/consoleFull)** for PR 14713 at commit [`82935a7`](https://github.com/apache/spark/commit/

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14279 Sigh.. I will proceed but please let me note that this will also bring a breaking change for reading as well not only writing.. It seems there is a backwards compatibility problem with

[GitHub] spark issue #14709: [SPARK-17150][SQL] Support SQL generation for inline tab...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14709 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 e

[GitHub] spark issue #14709: [SPARK-17150][SQL] Support SQL generation for inline tab...

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

[GitHub] spark issue #14709: [SPARK-17150][SQL] Support SQL generation for inline tab...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14709 **[Test build #64054 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64054/consoleFull)** for PR 14709 at commit [`442918f`](https://github.com/apache/spark/commit/

[GitHub] spark pull request #14682: [SPARK-17104][SQL] LogicalRelation.newInstance sh...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14682#discussion_r75452031 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala --- @@ -79,11 +79,18 @@ case class LogicalRelation(

[GitHub] spark pull request #14155: [SPARK-16498][SQL] move hive hack for data source...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14155#discussion_r75451232 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala --- @@ -144,16 +161,147 @@ private[spark] class HiveExternalCatalog(c

[GitHub] spark pull request #14155: [SPARK-16498][SQL] move hive hack for data source...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14155#discussion_r75451088 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala --- @@ -144,16 +161,147 @@ private[spark] class HiveExternalCatalog(c

[GitHub] spark pull request #14597: [SPARK-17017][MLLIB][ML] add a chiSquare Selector...

2016-08-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/14597#discussion_r75451038 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/ChiSqSelector.scala --- @@ -189,11 +228,35 @@ class ChiSqSelector @Since("1.3.0") ( */

[GitHub] spark issue #14643: [SPARK-17057][ML] ProbabilisticClassifierModels' predict...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14643 Thresholds are just that -- thresholds. The meaning is certainly as in https://github.com/apache/spark/pull/14643#discussion_r75290480 While I kind of like the idea of also treating them as a 'weigh

[GitHub] spark issue #14155: [SPARK-16498][SQL] move hive hack for data source table ...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14155 **[Test build #64058 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64058/consoleFull)** for PR 14155 at commit [`6ca8909`](https://github.com/apache/spark/commit/6

[GitHub] spark pull request #14155: [SPARK-16498][SQL] move hive hack for data source...

2016-08-19 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14155#discussion_r75450667 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -584,13 +579,8 @@ case class AlterTableSetLocationCommand(

[GitHub] spark issue #14279: [SPARK-16216][SQL] Read/write timestamps and dates in IS...

2016-08-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14279 Yup, thanks for informing me. --- 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 #14551: [SPARK-16961][CORE] Fixed off-by-one error that b...

2016-08-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14551 --- 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 issue #14551: [SPARK-16961][CORE] Fixed off-by-one error that biased r...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14551 Merged to master/2.0 --- 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,

[GitHub] spark pull request #14712: [SPARK-17072] [SQL] support table-level statistic...

2016-08-19 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r75450024 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -95,6 +95,12 @@ abstract class LogicalPlan extends

[GitHub] spark issue #14711: [SPARK-16822] [DOC] [Support latex in scaladoc with Math...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14711 I'm confused, isn't this related to SPARK-17095? you're already fixing mathjax issues there. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #14686: [SPARK-16253][SQL] make spark sql compatible with hive s...

2016-08-19 Thread kekdke
Github user kekdke commented on the issue: https://github.com/apache/spark/pull/14686 yes ,i have try it ,but the error is still "execution.ScriptTransformationWriterThread: /bin/bash: test1.py: command not found" could i add your QQ? --- If your project is set up for it, you

[GitHub] spark issue #14715: [SPARK-17085] [Streaming] [Documentation and actual code...

2016-08-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14715 That sounds correct given https://github.com/apache/spark/blob/5545b791096756b07b3207fb3de13b68b9a37b00/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationChecker.s

[GitHub] spark issue #14712: [SPARK-17072] [SQL] support table-level statistics gener...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14712 **[Test build #3227 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3227/consoleFull)** for PR 14712 at commit [`4375e76`](https://github.com/apache/spark/commit

[GitHub] spark pull request #14181: [SPARK-15382][SQL] Fix a rule to push down projec...

2016-08-19 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/14181#discussion_r75448687 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -148,6 +148,21 @@ class SimpleTestOptimizer extends Opti

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

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

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

2016-08-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14713 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 e

[GitHub] spark issue #14713: [SPARK-16994][SQL] Whitelist operators for predicate pus...

2016-08-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14713 **[Test build #64050 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64050/consoleFull)** for PR 14713 at commit [`9be428a`](https://github.com/apache/spark/commit/

<    1   2   3   4   5   6   7   >