[GitHub] spark pull request #16373: [SPARK-18961][SQL] Support `SHOW TABLE EXTENDED ....

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16373#discussion_r94004059 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala --- @@ -619,18 +621,34 @@ case class ShowTablesCommand( //

[GitHub] spark issue #15996: [SPARK-18567][SQL] Simplify CreateDataSourceTableAsSelec...

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

[GitHub] spark issue #15996: [SPARK-18567][SQL] Simplify CreateDataSourceTableAsSelec...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15996 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #15996: [SPARK-18567][SQL] Simplify CreateDataSourceTableAsSelec...

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

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution by dedupli...

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

[GitHub] spark pull request #14452: [SPARK-16849][SQL] Improve subquery execution by ...

2016-12-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/14452#discussion_r94001153 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1167,3 +1173,211 @@ object

[GitHub] spark issue #16419: [MINOR][DOC]Fix doc of ForeachWriter to use writeStream

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

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r94001020 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -60,23 +60,27 @@ class

[GitHub] spark pull request #16419: [MINOR][DOC]Fix doc of ForeachWriter to use write...

2016-12-27 Thread carsonwang
GitHub user carsonwang opened a pull request: https://github.com/apache/spark/pull/16419 [MINOR][DOC]Fix doc of ForeachWriter to use writeStream ## What changes were proposed in this pull request? Fix the document of `ForeachWriter` to use `writeStream` instead of `write`

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r94000876 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala --- @@ -211,6 +211,52 @@ object JdbcUtils extends

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r94000781 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala --- @@ -211,6 +211,52 @@ object JdbcUtils extends

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r94000757 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala --- @@ -211,6 +211,52 @@ object JdbcUtils extends

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

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

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r94000523 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala --- @@ -211,6 +211,52 @@ object JdbcUtils extends

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r93999899 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcRelationProvider.scala --- @@ -60,23 +60,27 @@ class

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16364: [DOC][BUILD][MINOR] add doc on new make-distribution swi...

2016-12-27 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/16364 thanks @srowen - merged to master. we have currently info on R packaging at https://github.com/apache/spark/blob/master/R/CRAN_RELEASE.md - from a quick check we don't have any existing

[GitHub] spark pull request #15664: [SPARK-18123][SQL] Use db column names instead of...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/15664#discussion_r93999377 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala --- @@ -211,6 +211,52 @@ object JdbcUtils extends

[GitHub] spark pull request #16364: [DOC][BUILD][MINOR] add doc on new make-distribut...

2016-12-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/16364 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-27 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/16410 Aha, okay and I'll fix that way. 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 not have this feature

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-27 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/16410 `create table t(a int, b int) partitioned by (a)`, the schema of table t is: . This behavior is intentional and already published, we can not change it. What we should do is to find out

[GitHub] spark pull request #16404: [SPARK-18969][SQL] Support grouping by nondetermi...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16404#discussion_r93998144 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1918,28 +1918,37 @@ class Analyzer( case

[GitHub] spark pull request #16404: [SPARK-18969][SQL] Support grouping by nondetermi...

2016-12-27 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/16404#discussion_r93997721 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1918,28 +1918,37 @@ class Analyzer( case

[GitHub] spark pull request #16404: [SPARK-18969][SQL] Support grouping by nondetermi...

2016-12-27 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/16404#discussion_r93997700 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1918,28 +1918,37 @@ class Analyzer( case

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-27 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13909 yea we should! thanks for pointing this out! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread jiangxb1987
Github user jiangxb1987 commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93996591 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,121 @@ class Analyzer( *

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread jiangxb1987
Github user jiangxb1987 commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93996552 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala --- @@ -543,4 +545,34 @@ class SQLViewSuite extends QueryTest

[GitHub] spark issue #15996: [SPARK-18567][SQL] Simplify CreateDataSourceTableAsSelec...

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

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93996247 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,121 @@ class Analyzer( *

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93995925 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,121 @@ class Analyzer( *

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93995903 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93995882 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLViewSuite.scala --- @@ -543,4 +545,34 @@ class SQLViewSuite extends QueryTest

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93995592 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -377,6 +378,36 @@ case class

[GitHub] spark issue #14452: [SPARK-16849][SQL] Improve subquery execution by dedupli...

2016-12-27 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/14452 Q47 is a typical query that can benefit from this PR: with v1 as( select i_category, i_brand, s_store_name, s_company_name, d_year, d_moy,

[GitHub] spark issue #13326: [SPARK-15560] [Mesos] Queued/Supervise drivers waiting f...

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

[GitHub] spark issue #13326: [SPARK-15560] [Mesos] Queued/Supervise drivers waiting f...

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

[GitHub] spark issue #13326: [SPARK-15560] [Mesos] Queued/Supervise drivers waiting f...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13326 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #13326: [SPARK-15560] [Mesos] Queued/Supervise drivers waiting f...

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

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread jiangxb1987
Github user jiangxb1987 commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93994361 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -377,6 +378,36 @@ case class

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16291 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93993539 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -377,6 +378,36 @@ case class

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark pull request #16393: [SPARK-18993] [Build] Revert Split test-tags into...

2016-12-27 Thread gatorsmile
Github user gatorsmile closed the pull request at: https://github.com/apache/spark/pull/16393 --- 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

[GitHub] spark issue #16418: [SPARK-18993][BUILD] Unable to build/compile Spark in In...

2016-12-27 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16418 This is the workaround I did in my local environment. It works well. 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

[GitHub] spark issue #9759: [SPARK-11753][SQL][test-hadoop2.2] Make allowNonNumericNu...

2016-12-27 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/9759 @srowen Unfortunately, this fixing is only included since 2.8.1. Even latest maintenance release 2.7.8 doesn't contain it. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request #16415: [SPARK-19007]Speedup and optimize the GradientBoo...

2016-12-27 Thread zdh2292390
Github user zdh2292390 commented on a diff in the pull request: https://github.com/apache/spark/pull/16415#discussion_r93988463 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/impl/GradientBoostedTrees.scala --- @@ -329,8 +330,16 @@ private[spark] object

[GitHub] spark pull request #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftSer...

2016-12-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/16392 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark issue #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftServer.sin...

2016-12-27 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/16392 LGTM, merging 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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16291 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark pull request #16415: [SPARK-19007]Speedup and optimize the GradientBoo...

2016-12-27 Thread zdh2292390
Github user zdh2292390 commented on a diff in the pull request: https://github.com/apache/spark/pull/16415#discussion_r93987658 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/impl/GradientBoostedTrees.scala --- @@ -309,6 +309,7 @@ private[spark] object

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16403: [SPARK-18819][CORE] Double byte alignment on ARM platfor...

2016-12-27 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/16403 While we ideally expect `static final` is evaluated as a constant by a JIT compiler, I do not know the fact. I would appreciate someone would look at the binary code generated by OpenJDK since I

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16291 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

2016-12-27 Thread sitalkedia
Github user sitalkedia commented on the issue: https://github.com/apache/spark/pull/16291 jenkins 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 feature enabled and

[GitHub] spark issue #13143: [SPARK-15359] [Mesos] Mesos dispatcher should handle DRI...

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

[GitHub] spark issue #13143: [SPARK-15359] [Mesos] Mesos dispatcher should handle DRI...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13143 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #13143: [SPARK-15359] [Mesos] Mesos dispatcher should handle DRI...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16291 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #16291: [SPARK-18838][CORE] Use separate executor service for ea...

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

[GitHub] spark issue #13143: [SPARK-15359] [Mesos] Mesos dispatcher should handle DRI...

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

[GitHub] spark issue #16370: [SPARK-18960][SQL][SS] Avoid double reading file which i...

2016-12-27 Thread uncleGen
Github user uncleGen commented on the issue: https://github.com/apache/spark/pull/16370 @srowen It looks like @zsxwing has no time to feed back. --- 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 #16255: [SPARK-18609][SQL]Fix when CTE with Join between ...

2016-12-27 Thread windpiger
Github user windpiger commented on a diff in the pull request: https://github.com/apache/spark/pull/16255#discussion_r93892249 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -200,6 +200,8 @@ object RemoveAliasOnlyProject

[GitHub] spark issue #16411: [SPARK-17984][YARN][Mesos][Deploy][WIP] add executor lau...

2016-12-27 Thread xiaochang-wu
Github user xiaochang-wu commented on the issue: https://github.com/apache/spark/pull/16411 @srowen do you mean #15579? 1) Fix some character escape issues of #15579 if the command string contains some special characters like ' 2) Add Standalone and Mesos support as #15579

[GitHub] spark issue #13323: [SPARK-15555] [Mesos] Driver with --supervise option can...

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

[GitHub] spark issue #13323: [SPARK-15555] [Mesos] Driver with --supervise option can...

2016-12-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13323 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #13323: [SPARK-15555] [Mesos] Driver with --supervise option can...

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

[GitHub] spark pull request #16403: [SPARK-18819][CORE] Double byte alignment on ARM ...

2016-12-27 Thread michaelkamprath
Github user michaelkamprath commented on a diff in the pull request: https://github.com/apache/spark/pull/16403#discussion_r93983319 --- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java --- @@ -244,6 +257,34 @@ public static void throwException(Throwable t)

[GitHub] spark issue #16386: [SPARK-18352][SQL] Support parsing multiline json files

2016-12-27 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16386 Only regarding the comment, https://github.com/apache/spark/pull/16386#issuecomment-269386229, I have a similar (rather combined) idea that we provide another option such as corrupt file name

[GitHub] spark pull request #16403: [SPARK-18819][CORE] Double byte alignment on ARM ...

2016-12-27 Thread michaelkamprath
Github user michaelkamprath commented on a diff in the pull request: https://github.com/apache/spark/pull/16403#discussion_r93982964 --- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java --- @@ -119,7 +127,12 @@ public static void putFloat(Object object, long

[GitHub] spark pull request #16403: [SPARK-18819][CORE] Double byte alignment on ARM ...

2016-12-27 Thread michaelkamprath
Github user michaelkamprath commented on a diff in the pull request: https://github.com/apache/spark/pull/16403#discussion_r93982937 --- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java --- @@ -244,6 +257,34 @@ public static void throwException(Throwable t)

[GitHub] spark pull request #16403: [SPARK-18819][CORE] Double byte alignment on ARM ...

2016-12-27 Thread michaelkamprath
Github user michaelkamprath commented on a diff in the pull request: https://github.com/apache/spark/pull/16403#discussion_r93982849 --- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java --- @@ -44,6 +49,9 @@ public static final int

[GitHub] spark issue #13323: [SPARK-15555] [Mesos] Driver with --supervise option can...

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

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93974759 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93974324 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93981206 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93982097 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93980975 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93974665 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93973973 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93972115 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93972699 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93974634 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93973211 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93972268 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93973041 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93982121 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93982401 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93979950 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93975729 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93975041 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #15211: [SPARK-14709][ML] spark.ml API for linear SVM

2016-12-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/15211#discussion_r93981539 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala --- @@ -0,0 +1,558 @@ +/* + * Licensed to the Apache Software

  1   2   3   >