[GitHub] spark issue #14793: [SPARK-17221] [SQL] Build File-based Test Cases for Usin...

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

[GitHub] spark issue #14793: [SPARK-17221] [SQL] Build File-based Test Cases for Usin...

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

[GitHub] spark pull request #14926: [SPARK-17365][Core] Remove/Kill multiple executor...

2016-09-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/14926#discussion_r77309343 --- Diff: core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala --- @@ -279,14 +280,18 @@ private[spark] class ExecutorAllocationManager(

[GitHub] spark pull request #14926: [SPARK-17365][Core] Remove/Kill multiple executor...

2016-09-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/14926#discussion_r77309320 --- Diff: core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala --- @@ -279,14 +280,18 @@ private[spark] class ExecutorAllocationManager(

[GitHub] spark pull request #14762: [SPARK-16962][CORE][SQL] Fix misaligned record ac...

2016-09-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/14762#discussion_r77308494 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/compression/CompressibleColumnBuilder.scala --- @@ -61,8 +63,12 @@

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

2016-09-02 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77308279 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala --- @@ -383,21 +384,34 @@ private[spark] class HiveExternalCatalog(conf:

[GitHub] spark pull request #14733: [SPARK-17170] [SQL] InMemoryTableScanExec driver-...

2016-09-02 Thread pwoody
Github user pwoody commented on a diff in the pull request: https://github.com/apache/spark/pull/14733#discussion_r77308047 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala --- @@ -142,13 +171,16 @@ case class

[GitHub] spark pull request #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveCl...

2016-09-02 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14888#discussion_r77307928 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -240,54 +240,49 @@ case class

[GitHub] spark issue #14891: [SQL][DOC][MINOR] Add (Scala-specific) and (Java-specifi...

2016-09-02 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14891 @srowen, anyone you'd recommend to accept the PR (after you accepted)? --- 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 #14712: [SPARK-17072] [SQL] support table-level statistic...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77307355 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -130,6 +130,7 @@ case class CatalogTable(

[GitHub] spark issue #14873: [SPARK-17308]Improved the spark core code by replacing a...

2016-09-02 Thread shiv4nsh
Github user shiv4nsh commented on the issue: https://github.com/apache/spark/pull/14873 @srowen : I have updated the PR according to your comments please review it ! Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

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

2016-09-02 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77307095 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -88,24 +85,54 @@ case class

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

2016-09-02 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77306895 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -130,6 +130,7 @@ case class CatalogTable(

[GitHub] spark pull request #14883: [SPARK-17319] [SQL] Move addJar from HiveSessionS...

2016-09-02 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14883#discussion_r77306344 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala --- @@ -184,4 +184,17 @@ abstract class

[GitHub] spark issue #14929: [SPARK-17374][SQL] Better error messages when parsing JS...

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

[GitHub] spark issue #14929: [SPARK-17374][SQL] Better error messages when parsing JS...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14929 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 #14900: [WEBUI][SPARK-17342] Style of event timeline is b...

2016-09-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14900 --- 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 #14929: [SPARK-17374][SQL] Better error messages when parsing JS...

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

[GitHub] spark issue #14900: [WEBUI][SPARK-17342] Style of event timeline is broken

2016-09-02 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14900 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 #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveCl...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14888#discussion_r77305123 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -240,54 +240,49 @@ case class InsertIntoHiveTable(

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark pull request #14762: [SPARK-16962][CORE][SQL] Fix misaligned record ac...

2016-09-02 Thread sumansomasundar
Github user sumansomasundar commented on a diff in the pull request: https://github.com/apache/spark/pull/14762#discussion_r77304811 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/compression/CompressibleColumnBuilder.scala --- @@ -61,8 +63,12 @@

[GitHub] spark pull request #14929: [SPARK-17374][SQL] Better error messages when par...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14929#discussion_r77304677 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala --- @@ -62,8 +68,39 @@ class JacksonParser(

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14850 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 #14873: [SPARK-17308]Improved the spark core code by repl...

2016-09-02 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/14873#discussion_r77304359 --- Diff: repl/scala-2.11/src/main/scala/org/apache/spark/repl/Main.scala --- @@ -44,7 +44,9 @@ object Main extends Logging { private def

[GitHub] spark pull request #14929: [SPARK-17374][SQL] Better error messages when par...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14929#discussion_r77304426 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala --- @@ -52,6 +53,11 @@ class JacksonParser(

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14712 looks pretty good now! I left some comments about some small issues, thanks for working on it! --- If your project is set up for it, you can reply to this email and have your reply appear on

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77304066 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -88,24 +85,54 @@ case class

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14873: [SPARK-17308]Improved the spark core code by replacing a...

2016-09-02 Thread shiv4nsh
Github user shiv4nsh commented on the issue: https://github.com/apache/spark/pull/14873 @srowen : Do we have to retest this build as I have done the changes implied by Byran ! --- 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 #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14913 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 #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77303607 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala --- @@ -610,6 +637,14 @@ object HiveExternalCatalog { val

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77303388 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala --- @@ -383,21 +384,34 @@ private[spark] class

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14913 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 #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14913: [SPARK-17358][SQL] Cached table(parquet/orc) should be s...

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

[GitHub] spark issue #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14834 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 #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

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

[GitHub] spark issue #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

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

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77302422 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala --- @@ -236,7 +236,8 @@ object DataSourceStrategy

[GitHub] spark pull request #14883: [SPARK-17319] [SQL] Move addJar from HiveSessionS...

2016-09-02 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/14883#discussion_r77302207 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala --- @@ -184,4 +184,17 @@ abstract class ExternalCatalog {

[GitHub] spark issue #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

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

[GitHub] spark issue #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14834 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 #14834: [SPARK-17163][ML][WIP] Unified LogisticRegression interf...

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

[GitHub] spark issue #14929: [SPARK-17374][SQL] Better error messages when parsing JS...

2016-09-02 Thread clockfly
Github user clockfly commented on the issue: https://github.com/apache/spark/pull/14929 @cloud-fan --- 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

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77301653 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -88,24 +85,54 @@ case class

[GitHub] spark issue #14929: [SPARK-17374][SQL] Better error messages when parsing JS...

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

[GitHub] spark pull request #14915: [SPARK-17356][SQL] Fix out of memory issue when g...

2016-09-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/14915#discussion_r77301211 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -604,6 +604,7 @@ abstract class TreeNode[BaseType <:

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77301097 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -88,24 +85,54 @@ case class

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark pull request #14921: [SPARK-17361][SQL] file-based external table with...

2016-09-02 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/14921#discussion_r77300888 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala --- @@ -314,12 +314,8 @@ case class DataSource( /**

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

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14712#discussion_r77300889 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -130,6 +130,7 @@ case class CatalogTable(

[GitHub] spark pull request #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveCl...

2016-09-02 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14888#discussion_r77300824 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -240,54 +240,49 @@ case class

[GitHub] spark pull request #14929: [Don't merge][WIP] Better error message for JSON ...

2016-09-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/14929#discussion_r77300596 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/json/JacksonParser.scala --- @@ -62,8 +68,35 @@ class JacksonParser(

[GitHub] spark pull request #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveCl...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14888#discussion_r77300500 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -240,54 +240,49 @@ case class InsertIntoHiveTable(

[GitHub] spark pull request #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveCl...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14888#discussion_r77300402 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -240,54 +240,49 @@ case class InsertIntoHiveTable(

[GitHub] spark issue #14934: [SPARKR][DOC] regexp_extract should doc that it returns ...

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

[GitHub] spark issue #14934: [SPARKR][DOC] regexp_extract should doc that it returns ...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14934 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 #14934: [SPARKR][DOC] regexp_extract should doc that it returns ...

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

[GitHub] spark issue #14793: [SPARK-17221] [SQL] Build File-based Test Cases for Usin...

2016-09-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/14793 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

[GitHub] spark issue #14613: [SPARK-16883][SparkR]:SQL decimal type is not properly c...

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

[GitHub] spark issue #14613: [SPARK-16883][SparkR]:SQL decimal type is not properly c...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14613 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 #14850: [SPARK-17279][SQL] better error message for exceptions d...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14850 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 #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark issue #14850: [SPARK-17279][SQL] better error message for exceptions d...

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

[GitHub] spark issue #14613: [SPARK-16883][SparkR]:SQL decimal type is not properly c...

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

[GitHub] spark issue #14921: [SPARK-17361][SQL] file-based external table without pat...

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

[GitHub] spark issue #14921: [SPARK-17361][SQL] file-based external table without pat...

2016-09-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14921 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 #14433: [SPARK-16829][SparkR]:sparkR sc.setLogLevel doesn't work

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

[GitHub] spark issue #14929: [Don't merge][WIP] Better error message for JSON file fo...

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

[GitHub] spark issue #14921: [SPARK-17361][SQL] file-based external table without pat...

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

[GitHub] spark issue #14433: [SPARK-16829][SparkR]:sparkR sc.setLogLevel doesn't work

2016-09-02 Thread wangmiao1981
Github user wangmiao1981 commented on the issue: https://github.com/apache/spark/pull/14433 @shivaram I simplified the solution by only changing the message as we discussed. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77298682 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14793: [SPARK-17221] [SQL] Build File-based Test Cases for Usin...

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

[GitHub] spark issue #14793: [SPARK-17221] [SQL] Build File-based Test Cases for Usin...

2016-09-02 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/14793 cc @cloud-fan @clockfly --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77298211 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14521: [SPARK-16935] [SQL] Verification of Function-related Ext...

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

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77298118 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297969 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297902 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14934: [SPARKR][DOC] regexp_extract should doc that it returns ...

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

[GitHub] spark pull request #14934: [SPARKR][DOC] regexp_extract should doc that it r...

2016-09-02 Thread felixcheung
GitHub user felixcheung opened a pull request: https://github.com/apache/spark/pull/14934 [SPARKR][DOC] regexp_extract should doc that it returns empty string when match fails ## What changes were proposed in this pull request? Doc change - see

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297807 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14888: [SPARK-17324] [SQL] Remove Direct Usage of HiveClient in...

2016-09-02 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/14888 cc @cloud-fan @yhuai --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297662 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297537 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14613: [SPARK-16883][SparkR]:SQL decimal type is not properly c...

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

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297372 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297304 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297192 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #14079: [SPARK-8425][CORE] New Blacklist Mechanism

2016-09-02 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/14079#discussion_r77297069 --- Diff: core/src/test/scala/org/apache/spark/scheduler/BlacklistTrackerSuite.scala --- @@ -0,0 +1,503 @@ +/* + * Licensed to the Apache

<    1   2   3   4   5