[GitHub] [spark] cloud-fan commented on a change in pull request #30079: [SPARK-33174][SQL] Migrate DROP TABLE to use UnresolvedTableOrView to resolve the identifier

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30079: URL: https://github.com/apache/spark/pull/30079#discussion_r511821763 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveSessionCatalog.scala ## @@ -368,9 +368,13 @@ class

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716420043 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716419788 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716420043 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] cloud-fan commented on a change in pull request #30142: [SPARK-33095] Follow up, support alter table column rename.

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30142: URL: https://github.com/apache/spark/pull/30142#discussion_r511815463 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ## @@ -2858,6 +2858,14 @@ object SQLConf { .stringConf

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716419244 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
SparkQA commented on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716419761 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34867/

[GitHub] [spark] SparkQA removed a comment on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
SparkQA removed a comment on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716352782 **[Test build #130265 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130265/testReport)** for PR 30146 at commit

[GitHub] [spark] AmplabJenkins commented on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716419788 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716419237 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] SparkQA commented on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
SparkQA commented on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716418995 **[Test build #130265 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130265/testReport)** for PR 30146 at commit

[GitHub] [spark] AmplabJenkins commented on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716419237 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] cloud-fan commented on a change in pull request #30142: [SPARK-33095] Follow up, support alter table column rename.

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30142: URL: https://github.com/apache/spark/pull/30142#discussion_r511814553 ## File path: external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MySQLIntegrationSuite.scala ## @@ -77,11 +86,26 @@ class

[GitHub] [spark] SparkQA commented on pull request #29421: [SPARK-32388][SQL] TRANSFORM with schema-less mode should keep the same with hive

2020-10-26 Thread GitBox
SparkQA commented on pull request #29421: URL: https://github.com/apache/spark/pull/29421#issuecomment-716418662 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34868/

[GitHub] [spark] cloud-fan commented on a change in pull request #30142: [SPARK-33095] Follow up, support alter table column rename.

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30142: URL: https://github.com/apache/spark/pull/30142#discussion_r511814190 ## File path: external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MySQLIntegrationSuite.scala ## @@ -77,11 +86,26 @@ class

[GitHub] [spark] SparkQA commented on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
SparkQA commented on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716417932 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34869/

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716417391 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716417391 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716417357 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34866/

[GitHub] [spark] cloud-fan commented on a change in pull request #30093: [SPARK-33183][SQL] Fix EliminateSorts bug when removing global sorts

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30093: URL: https://github.com/apache/spark/pull/30093#discussion_r511810245 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1052,12 +1052,18 @@ object CombineFilters

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716412379 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716412371 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] SparkQA removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
SparkQA removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716352760 **[Test build #130264 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130264/testReport)** for PR 30147 at commit

[GitHub] [spark] SparkQA commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
SparkQA commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716412135 **[Test build #130264 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130264/testReport)** for PR 30147 at commit

[GitHub] [spark] AmplabJenkins commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716412371 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716393458 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] dongjoon-hyun closed pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun closed pull request #30066: URL: https://github.com/apache/spark/pull/30066 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [spark] dongjoon-hyun commented on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716404226 Thank you, @HyukjinKwon and @maropu . GitHub Action SparkR passed. I'll merge this. This is an

[GitHub] [spark] cloud-fan commented on a change in pull request #29970: [SPARK-33087][SQL] DataFrameWriterV2 should delegate table resolution to the analyzer

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #29970: URL: https://github.com/apache/spark/pull/29970#discussion_r511797749 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/v2Commands.scala ## @@ -59,7 +61,9 @@ case class AppendData(

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716397147 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on a change in pull request #29970: [SPARK-33087][SQL] DataFrameWriterV2 should delegate table resolution to the analyzer

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #29970: URL: https://github.com/apache/spark/pull/29970#discussion_r511797749 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/v2Commands.scala ## @@ -59,7 +61,9 @@ case class AppendData(

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716395189 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716402329 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA commented on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
SparkQA commented on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716403050 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34867/

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716397130 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716402283 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34866/

[GitHub] [spark] AmplabJenkins commented on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716402329 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
SparkQA commented on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716402308 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34865/

[GitHub] [spark] SparkQA commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
SparkQA commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716397107 Kubernetes integration test status failure URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34864/

[GitHub] [spark] AmplabJenkins commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716397130 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716395189 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716395161 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34863/

[GitHub] [spark] cloud-fan commented on a change in pull request #30089: [SPARK-33137][SQL] Support ALTER TABLE in JDBC v2 Table Catalog: update type and nullability of columns (Postgres dialect)

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30089: URL: https://github.com/apache/spark/pull/30089#discussion_r511787948 ## File path: external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/V2JDBCTest.scala ## @@ -73,6 +73,30 @@ trait V2JDBCTest

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716393444 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716393444 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511786935 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisTest.scala ## @@ -53,16 +53,18 @@ trait AnalysisTest extends

[GitHub] [spark] SparkQA removed a comment on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
SparkQA removed a comment on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716388196 **[Test build #130274 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130274/testReport)** for PR 29820 at commit

[GitHub] [spark] SparkQA commented on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
SparkQA commented on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716393413 **[Test build #130274 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130274/testReport)** for PR 29820 at commit

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511786254 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/PruneHiveTablePartitions.scala ## @@ -29,22 +29,9 @@ import

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511785672 ## File path: sql/core/src/test/scala/org/apache/spark/sql/sources/DataSourceAnalysisSuite.scala ## @@ -51,21 +51,26 @@ class DataSourceAnalysisSuite

[GitHub] [spark] leanken commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
leanken commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511785233 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/PruneHiveTablePartitions.scala ## @@ -29,22 +29,9 @@ import

[GitHub] [spark] leanken commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
leanken commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511784684 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisTest.scala ## @@ -53,16 +53,18 @@ trait AnalysisTest extends

[GitHub] [spark] SparkQA commented on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
SparkQA commented on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716391455 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34865/

[GitHub] [spark] leanken commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
leanken commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511782477 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/Columnar.scala ## @@ -439,7 +438,7 @@ case class RowToColumnarExec(child: SparkPlan)

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511781964 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/Columnar.scala ## @@ -439,7 +438,7 @@ case class RowToColumnarExec(child:

[GitHub] [spark] leanken commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
leanken commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511781341 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala ## @@ -771,22 +771,23 @@ class AnalysisSuite

[GitHub] [spark] SparkQA commented on pull request #29820: [WIP][SPARK-20044][UI] Support Spark UI behind front-end reverse proxy using a path prefix Revert proxy url

2020-10-26 Thread GitBox
SparkQA commented on pull request #29820: URL: https://github.com/apache/spark/pull/29820#issuecomment-716388196 **[Test build #130274 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130274/testReport)** for PR 29820 at commit

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511780792 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/SubstituteUnresolvedOrdinalsSuite.scala ## @@ -36,31 +37,40 @@ class

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511780716 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/SubstituteUnresolvedOrdinalsSuite.scala ## @@ -36,31 +37,40 @@ class

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511779789 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisTest.scala ## @@ -53,16 +53,18 @@ trait AnalysisTest extends

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511779442 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala ## @@ -771,22 +771,23 @@ class AnalysisSuite

[GitHub] [spark] cloud-fan commented on a change in pull request #30097: [SPARK-33140][SQL] remove SQLConf and SparkSession in all sub-class of Rule[QueryPlan]

2020-10-26 Thread GitBox
cloud-fan commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511778077 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -255,19 +255,19 @@ class Analyzer(

[GitHub] [spark] SparkQA commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
SparkQA commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716384910 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34864/

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716384398 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34863/

[GitHub] [spark] AmplabJenkins commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716383286 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716383286 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716383275 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34861/

[GitHub] [spark] dongjoon-hyun commented on pull request #30123: [SPARK-33234][INFRA] Generates SHA-512 using shasum

2020-10-26 Thread GitBox
dongjoon-hyun commented on pull request #30123: URL: https://github.com/apache/spark/pull/30123#issuecomment-716382977 Finally, I assigned SPARK-33234 to you. Thank you! This is an automated message from the Apache Git

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716381964 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716381964 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

2020-10-26 Thread GitBox
SparkQA commented on pull request #30138: URL: https://github.com/apache/spark/pull/30138#issuecomment-716381949 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34862/

[GitHub] [spark] maropu commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
maropu commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511773882 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716379436 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716379426 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] emilianbold commented on pull request #30123: [SPARK-33234][INFRA] Generates SHA-512 using shasum

2020-10-26 Thread GitBox
emilianbold commented on pull request #30123: URL: https://github.com/apache/spark/pull/30123#issuecomment-716379574 Hm, this is what JIRA says: Here are the details of your account: - Username: emi

[GitHub] [spark] SparkQA removed a comment on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
SparkQA removed a comment on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716356604 **[Test build #130269 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130269/testReport)** for PR 29414 at commit

[GitHub] [spark] AmplabJenkins commented on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716379426 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #29414: [SPARK-32106][SQL] Implement script transform in sql/core

2020-10-26 Thread GitBox
SparkQA commented on pull request #29414: URL: https://github.com/apache/spark/pull/29414#issuecomment-716379136 **[Test build #130269 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130269/testReport)** for PR 29414 at commit

[GitHub] [spark] SparkQA commented on pull request #24173: [SPARK-27237][SS] Introduce State schema validation among query restart

2020-10-26 Thread GitBox
SparkQA commented on pull request #24173: URL: https://github.com/apache/spark/pull/24173#issuecomment-716377176 **[Test build #130273 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130273/testReport)** for PR 24173 at commit

[GitHub] [spark] dongjoon-hyun commented on pull request #30123: [SPARK-33234][INFRA] Generates SHA-512 using shasum

2020-10-26 Thread GitBox
dongjoon-hyun commented on pull request #30123: URL: https://github.com/apache/spark/pull/30123#issuecomment-716376432 ? https://user-images.githubusercontent.com/9700541/97146674-78a15580-1725-11eb-8f1f-15d6e65cdd24.png;>

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511768101 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511768101 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30146: [SPARK-33241][SQL] Dynamic pruning on data column

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30146: URL: https://github.com/apache/spark/pull/30146#issuecomment-716361030 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA removed a comment on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA removed a comment on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716348937 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716369315 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716360611 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA commented on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
SparkQA commented on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716373195 **[Test build #130272 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130272/testReport)** for PR 30066 at commit

[GitHub] [spark] AmplabJenkins removed a comment on pull request #30131: [SPARK-33220][CORE]Use `scheduleWithFixedDelay` to avoid repeated unnecessary scheduling for a short time

2020-10-26 Thread GitBox
AmplabJenkins removed a comment on pull request #30131: URL: https://github.com/apache/spark/pull/30131#issuecomment-716364987 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] maropu commented on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
maropu commented on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716372391 (I'm not familiar with the R part though) the added GA script itself looks okay to me. This is an automated

[GitHub] [spark] maropu commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
maropu commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511765802 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511764083 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] dongjoon-hyun commented on pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on pull request #30066: URL: https://github.com/apache/spark/pull/30066#issuecomment-716371503 Thank you for review, @HyukjinKwon . This is an automated message from the Apache Git Service. To respond

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511764083 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] AmplabJenkins commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716370510 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716370033 **[Test build #130267 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/130267/testReport)** for PR 30143 at commit

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511764596 ## File path: .github/workflows/build_and_test.yml ## @@ -138,20 +136,6 @@ jobs: run: | python3.8 -m pip install numpy

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #30066: [SPARK-33239][INFRA] Use pre-built image at GitHub Action SparkR job

2020-10-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #30066: URL: https://github.com/apache/spark/pull/30066#discussion_r511764083 ## File path: .github/workflows/build_and_test.yml ## @@ -260,6 +244,68 @@ jobs: name: unit-tests-log-${{ matrix.modules

[GitHub] [spark] SparkQA commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
SparkQA commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716369296 Kubernetes integration test status success URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34859/

[GitHub] [spark] AmplabJenkins commented on pull request #30147: [SPARK-33240][SQL] Fail fast when fails to instantiate configured v2 session catalog

2020-10-26 Thread GitBox
AmplabJenkins commented on pull request #30147: URL: https://github.com/apache/spark/pull/30147#issuecomment-716369315 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #30143: [SPARK-32084][PYTHON][SQL] Expand dictionary functions

2020-10-26 Thread GitBox
SparkQA commented on pull request #30143: URL: https://github.com/apache/spark/pull/30143#issuecomment-716368205 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34861/

<    2   3   4   5   6   7   8   >