[GitHub] spark pull request #16422: [SPARK-17642] [SQL] support DESC EXTENDED/FORMATT...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16422#discussion_r126275554 --- Diff: sql/core/src/test/resources/sql-tests/results/describe-table-column.sql.out --- @@ -0,0 +1,133 @@ +-- Automatically generated by

[GitHub] spark pull request #16422: [SPARK-17642] [SQL] support DESC EXTENDED/FORMATT...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16422#discussion_r126275528 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -320,10 +320,15 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126275501 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -95,7 +93,6 @@ class SparkSessionBuilderSuite extends

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126275492 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -40,10 +39,10 @@ case class

[GitHub] spark pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126275485 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala --- @@ -42,17 +42,20 @@ case class

[GitHub] spark pull request #18307: [SPARK-21100][SQL] Add summary method as alternat...

2017-07-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/18307 --- 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 #18307: [SPARK-21100][SQL] Add summary method as alternative to ...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18307 thanks, 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 #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

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

[GitHub] spark issue #18566: [SPARK-21343] Refine the document for spark.reducer.maxR...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18566 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 #18566: [SPARK-21343] Refine the document for spark.reducer.maxR...

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

[GitHub] spark pull request #18566: [SPARK-21343] Refine the document for spark.reduc...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18566#discussion_r126275247 --- Diff: docs/configuration.md --- @@ -529,6 +529,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request #18566: [SPARK-21343] Refine the document for spark.reduc...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18566#discussion_r126275245 --- Diff: docs/configuration.md --- @@ -529,6 +529,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark issue #18566: [SPARK-21343] Refine the document for spark.reducer.maxR...

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

[GitHub] spark pull request #18566: [SPARK-21343] Refine the document for spark.reduc...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18566#discussion_r126275228 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -323,9 +323,10 @@ package object config { private[spark]

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 Now, each unit test becomes independent. --- 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 issue #18555: [Minor]add checkValue in spark.internal.config about how...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18555 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 #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126275145 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -36,18 +38,22 @@ class SparkSessionBuilderSuite extends

[GitHub] spark issue #18555: [Minor]add checkValue in spark.internal.config about how...

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

[GitHub] spark issue #18555: [Minor]add checkValue in spark.internal.config about how...

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

[GitHub] spark issue #16630: [SPARK-19270][ML] Add summary table to GLM summary

2017-07-07 Thread actuaryzhang
Github user actuaryzhang commented on the issue: https://github.com/apache/spark/pull/16630 @yanboliang Could you take a look? --- 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 #18393: [SPARK-20342][core] Update task accumulators before send...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18393 LGTM, pending test --- 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

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126274957 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -17,13 +17,15 @@ package org.apache.spark.sql

[GitHub] spark issue #18292: [SPARK-21083][SQL] Do not remove correct column stats wh...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18292 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 #18292: [SPARK-21083][SQL] Do not remove correct column stats wh...

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

[GitHub] spark issue #18292: [SPARK-21083][SQL] Do not remove correct column stats wh...

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

[GitHub] spark issue #18393: [SPARK-20342][core] Update task accumulators before send...

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

[GitHub] spark issue #18393: [SPARK-20342][core] Update task accumulators before send...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18393 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 #18393: [SPARK-20342][core] Update task accumulators before send...

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

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126274862 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -36,18 +38,22 @@ class SparkSessionBuilderSuite extends

[GitHub] spark issue #18393: [SPARK-20342][core] Update task accumulators before send...

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

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126274860 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -17,13 +17,15 @@ package org.apache.spark.sql

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126274671 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -17,13 +17,15 @@ package org.apache.spark.sql

[GitHub] spark pull request #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSession...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18567#discussion_r126274672 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala --- @@ -36,18 +38,22 @@ class SparkSessionBuilderSuite extends

[GitHub] spark issue #18568: [Revert][SPARK-21307][SQL] Remove SQLConf parameters fro...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18568 can we fix the active session problem? e.g. set active session in `SparkSession.sql` --- 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 #18562: [SPARK-21069][SS][DOCS] Add rate source to programming g...

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

[GitHub] spark issue #18562: [SPARK-21069][SS][DOCS] Add rate source to programming g...

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

[GitHub] spark issue #18562: [SPARK-21069][SS][DOCS] Add rate source to programming g...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18562 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 pull request #18292: [SPARK-21083][SQL] Do not remove correct column s...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18292#discussion_r126274515 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala --- @@ -42,17 +42,20 @@ case class

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18468 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 #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark pull request #16422: [SPARK-17642] [SQL] support DESC EXTENDED/FORMATT...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/16422#discussion_r126274465 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala --- @@ -626,6 +624,117 @@ case class DescribeTableCommand( }

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark pull request #16422: [SPARK-17642] [SQL] support DESC EXTENDED/FORMATT...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/16422#discussion_r126274424 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -320,10 +320,15 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark issue #18562: [SPARK-21069][SS][DOCS] Add rate source to programming g...

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

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274206 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274201 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274200 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274204 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274198 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274203 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -88,50 +137,76 @@ package object config { .createOptional

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274183 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -51,29 +63,66 @@ package object config {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274186 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -51,29 +63,66 @@ package object config {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126274185 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -51,29 +63,66 @@ package object config {

[GitHub] spark issue #18292: [SPARK-21083][SQL] Do not remove correct column stats wh...

2017-07-07 Thread wzhfy
Github user wzhfy commented on the issue: https://github.com/apache/spark/pull/18292 @cloud-fan @gatorsmile This pr is updated based on the lastest code. Would you take another look? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #16422: [SPARK-17642] [SQL] support DESC EXTENDED/FORMATTED tabl...

2017-07-07 Thread wzhfy
Github user wzhfy commented on the issue: https://github.com/apache/spark/pull/16422 @cloud-fan @gatorsmile I've updated this pr, and solved the qualified column name/nested column issue, could you please check again? --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request #18393: [SPARK-20342][core] Update task accumulators befo...

2017-07-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/18393#discussion_r126273879 --- Diff: core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala --- @@ -2277,6 +2277,29 @@ class DAGSchedulerSuite extends SparkFunSuite

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273805 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -93,30 +93,50 @@ class SparkSqlAstBuilder extends AstBuilder {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread heary-cao
Github user heary-cao commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273756 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -35,10 +35,22 @@ package object config {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread heary-cao
Github user heary-cao commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273760 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -35,10 +35,22 @@ package object config {

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273773 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -201,6 +202,193 @@ class StatisticsSuite extends

[GitHub] spark issue #18555: [Minor]add checkValue in spark.internal.config about how...

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

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

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

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

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

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273150 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -17,14 +17,17 @@ package

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273394 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273439 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273560 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273102 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -93,30 +93,50 @@ class SparkSqlAstBuilder extends AstBuilder {

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273584 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273555 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273031 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -93,30 +93,50 @@ class SparkSqlAstBuilder extends AstBuilder {

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273588 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273094 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -93,30 +93,50 @@ class SparkSqlAstBuilder extends AstBuilder {

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273582 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala --- @@ -181,6 +182,151 @@ class StatisticsSuite extends

[GitHub] spark pull request #18421: [SPARK-21213][SQL] Support collecting partition-l...

2017-07-07 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/18421#discussion_r126273260 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -72,3 +61,81 @@ case class AnalyzeTableCommand(

[GitHub] spark pull request #18300: [SPARK-21043][SQL] Add unionByName in Dataset

2017-07-07 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/18300#discussion_r126273573 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -112,6 +112,91 @@ class DataFrameSuite extends QueryTest with

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273432 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -51,29 +63,66 @@ package object config {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273429 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -51,29 +63,66 @@ package object config {

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273417 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -35,10 +35,22 @@ package object config {

[GitHub] spark issue #18459: [SPARK-13534][PYSPARK] Using Apache Arrow to increase pe...

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

[GitHub] spark issue #18459: [SPARK-13534][PYSPARK] Using Apache Arrow to increase pe...

2017-07-07 Thread shaneknapp
Github user shaneknapp commented on the issue: https://github.com/apache/spark/pull/18459 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 wishes so,

[GitHub] spark issue #18555: [Minor]add checkValue in spark.internal.config about how...

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

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273242 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -35,10 +35,22 @@ package object config {

[GitHub] spark issue #18394: [SPARK-20379][core] Allow SSL config to reference env va...

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

[GitHub] spark pull request #18555: [Minor]add checkValue in spark.internal.config ab...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18555#discussion_r126273196 --- Diff: core/src/main/scala/org/apache/spark/internal/config/package.scala --- @@ -35,10 +35,22 @@ package object config {

[GitHub] spark issue #18394: [SPARK-20379][core] Allow SSL config to reference env va...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18394 LGTM, pending tests --- 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

[GitHub] spark issue #18394: [SPARK-20379][core] Allow SSL config to reference env va...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18394 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

[GitHub] spark issue #18555: [Minor]add checkValue in spark.internal.config about how...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18555 ok to test --- 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 #18394: [SPARK-20379][core] Allow SSL config to reference...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18394#discussion_r126273151 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -373,6 +373,11 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with

[GitHub] spark pull request #18393: [SPARK-20342][core] Update task accumulators befo...

2017-07-07 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/18393#discussion_r126273069 --- Diff: core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala --- @@ -2277,6 +2277,29 @@ class DAGSchedulerSuite extends SparkFunSuite

[GitHub] spark issue #18459: [SPARK-13534][PYSPARK] Using Apache Arrow to increase pe...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18459 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 #18459: [SPARK-13534][PYSPARK] Using Apache Arrow to increase pe...

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

[GitHub] spark issue #18459: [SPARK-13534][PYSPARK] Using Apache Arrow to increase pe...

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

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

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

[GitHub] spark issue #18468: [SPARK-20873][SQL] Enhance ColumnVector to support compr...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18468 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 #18292: [SPARK-21083][SQL] Do not remove correct stats when re-a...

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

  1   2   3   4   5   >