[GitHub] [spark] maropu commented on a change in pull request #29199: [SPARK-32105][SQL][FOLLOWUP] Refactor current ScriptTransformationExec

2020-07-23 Thread GitBox
maropu commented on a change in pull request #29199: URL: https://github.com/apache/spark/pull/29199#discussion_r459250245 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ## @@ -1063,6 +1063,9 @@ private[hive] trait HiveInspectors {

[GitHub] [spark] maropu commented on a change in pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

2020-07-23 Thread GitBox
maropu commented on a change in pull request #29199: URL: https://github.com/apache/spark/pull/29199#discussion_r459250245 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ## @@ -1063,6 +1063,9 @@ private[hive] trait HiveInspectors {

[GitHub] [spark] SparkQA commented on pull request #29166: [SPARK-32280][SPARK-32372][SQL] ResolveReferences.dedupRight should only rewrite attributes for ancestor nodes of the conflict plan

2020-07-23 Thread GitBox
SparkQA commented on pull request #29166: URL: https://github.com/apache/spark/pull/29166#issuecomment-662851171 **[Test build #126375 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126375/testReport)** for PR 29166 at commit [`f73f186`](https://github.co

[GitHub] [spark] SparkQA commented on pull request #29014: [SPARK-32199][SPARK-32198] Reduce job failures during decommissioning

2020-07-23 Thread GitBox
SparkQA commented on pull request #29014: URL: https://github.com/apache/spark/pull/29014#issuecomment-662851164 **[Test build #126385 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126385/testReport)** for PR 29014 at commit [`7203980`](https://github.co

[GitHub] [spark] SparkQA removed a comment on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

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

[GitHub] [spark] AmplabJenkins commented on pull request #29107: [SPARK-32308][SQL] Move by-name resolution logic of unionByName from API code to analysis phase

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

[GitHub] [spark] AmplabJenkins commented on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

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

[GitHub] [spark] SparkQA commented on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

2020-07-23 Thread GitBox
SparkQA commented on pull request #29167: URL: https://github.com/apache/spark/pull/29167#issuecomment-662851174 **[Test build #126380 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126380/testReport)** for PR 29167 at commit [`477b696`](https://github.co

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

2020-07-23 Thread GitBox
SparkQA commented on pull request #29085: URL: https://github.com/apache/spark/pull/29085#issuecomment-662851159 **[Test build #126373 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126373/testReport)** for PR 29085 at commit [`7f3cff8`](https://github.co

[GitHub] [spark] SparkQA commented on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

2020-07-23 Thread GitBox
SparkQA commented on pull request #29199: URL: https://github.com/apache/spark/pull/29199#issuecomment-662851160 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] SparkQA commented on pull request #29192: [SPARK-32393][SQL] Fix postgres bpchar array support

2020-07-23 Thread GitBox
SparkQA commented on pull request #29192: URL: https://github.com/apache/spark/pull/29192#issuecomment-662851173 **[Test build #126374 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126374/testReport)** for PR 29192 at commit [`1000377`](https://github.co

[GitHub] [spark] SparkQA commented on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
SparkQA commented on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662851166 **[Test build #126390 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126390/testReport)** for PR 29188 at commit [`8a6d316`](https://github.co

[GitHub] [spark] AmplabJenkins commented on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

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

[GitHub] [spark] SparkQA commented on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

2020-07-23 Thread GitBox
SparkQA commented on pull request #29191: URL: https://github.com/apache/spark/pull/29191#issuecomment-662851161 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] SparkQA commented on pull request #29186: [SPARK-32386][SS][TESTS] Fix temp view leaking in Structured Streaming tests

2020-07-23 Thread GitBox
SparkQA commented on pull request #29186: URL: https://github.com/apache/spark/pull/29186#issuecomment-662851167 **[Test build #126379 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126379/testReport)** for PR 29186 at commit [`b654ebd`](https://github.co

[GitHub] [spark] SparkQA commented on pull request #29107: [SPARK-32308][SQL] Move by-name resolution logic of unionByName from API code to analysis phase

2020-07-23 Thread GitBox
SparkQA commented on pull request #29107: URL: https://github.com/apache/spark/pull/29107#issuecomment-662851158 **[Test build #126388 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126388/testReport)** for PR 29107 at commit [`2a9e1e4`](https://github.co

[GitHub] [spark] SparkQA removed a comment on pull request #29186: [SPARK-32386][SS][TESTS] Fix temp view leaking in Structured Streaming tests

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29186: URL: https://github.com/apache/spark/pull/29186#issuecomment-662819020 **[Test build #126379 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126379/testReport)** for PR 29186 at commit [`b654ebd`](https://gi

[GitHub] [spark] AmplabJenkins commented on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

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

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

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

[GitHub] [spark] SparkQA commented on pull request #29190: [do not review][testing master PR build][discard] comment change

2020-07-23 Thread GitBox
SparkQA commented on pull request #29190: URL: https://github.com/apache/spark/pull/29190#issuecomment-662851168 **[Test build #126387 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126387/testReport)** for PR 29190 at commit [`fa41817`](https://github.co

[GitHub] [spark] AmplabJenkins commented on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

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

[GitHub] [spark] AmplabJenkins commented on pull request #29190: [do not review][testing master PR build][discard] comment change

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

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

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29085: URL: https://github.com/apache/spark/pull/29085#issuecomment-662789794 **[Test build #126373 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126373/testReport)** for PR 29085 at commit [`7f3cff8`](https://gi

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

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

[GitHub] [spark] SparkQA removed a comment on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29167: URL: https://github.com/apache/spark/pull/29167#issuecomment-662819073 **[Test build #126380 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126380/testReport)** for PR 29167 at commit [`477b696`](https://gi

[GitHub] [spark] AmplabJenkins commented on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

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

[GitHub] [spark] SparkQA commented on pull request #29200: [SPARK-32251][SQL][TESTS][FOLLOWUP] improve SQL keyword test

2020-07-23 Thread GitBox
SparkQA commented on pull request #29200: URL: https://github.com/apache/spark/pull/29200#issuecomment-662851355 **[Test build #126391 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126391/testReport)** for PR 29200 at commit [`28b635b`](https://github.com

[GitHub] [spark] SparkQA removed a comment on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

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

[GitHub] [spark] SparkQA removed a comment on pull request #29014: [SPARK-32199][SPARK-32198] Reduce job failures during decommissioning

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29014: URL: https://github.com/apache/spark/pull/29014#issuecomment-662825676 **[Test build #126385 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126385/testReport)** for PR 29014 at commit [`7203980`](https://gi

[GitHub] [spark] SparkQA removed a comment on pull request #29166: [SPARK-32280][SPARK-32372][SQL] ResolveReferences.dedupRight should only rewrite attributes for ancestor nodes of the conflict plan

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29166: URL: https://github.com/apache/spark/pull/29166#issuecomment-662791379 **[Test build #126375 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126375/testReport)** for PR 29166 at commit [`f73f186`](https://gi

[GitHub] [spark] AmplabJenkins commented on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

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

[GitHub] [spark] SparkQA commented on pull request #28977: [SPARK-32389][TESTS] Add all hive.execution suite in the parallel test group

2020-07-23 Thread GitBox
SparkQA commented on pull request #28977: URL: https://github.com/apache/spark/pull/28977#issuecomment-662851376 **[Test build #126392 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126392/testReport)** for PR 28977 at commit [`6a68a7a`](https://github.com

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29107: [SPARK-32308][SQL] Move by-name resolution logic of unionByName from API code to analysis phase

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29199: URL: https://github.com/apache/spark/pull/29199#issuecomment-662851366 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 #29200: [SPARK-32251][SQL][TESTS][FOLLOWUP] improve SQL keyword test

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29200: URL: https://github.com/apache/spark/pull/29200#issuecomment-662848235 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 #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

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

[GitHub] [spark] SparkQA removed a comment on pull request #29107: [SPARK-32308][SQL] Move by-name resolution logic of unionByName from API code to analysis phase

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29107: URL: https://github.com/apache/spark/pull/29107#issuecomment-662840761 **[Test build #126388 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126388/testReport)** for PR 29107 at commit [`2a9e1e4`](https://gi

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29191: URL: https://github.com/apache/spark/pull/29191#issuecomment-662851237 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 #29192: [SPARK-32393][SQL] Fix postgres bpchar array support

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29192: URL: https://github.com/apache/spark/pull/29192#issuecomment-662791349 **[Test build #126374 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126374/testReport)** for PR 29192 at commit [`1000377`](https://gi

[GitHub] [spark] SparkQA removed a comment on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662843769 **[Test build #126390 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126390/testReport)** for PR 29188 at commit [`8a6d316`](https://gi

[GitHub] [spark] SparkQA removed a comment on pull request #29190: [do not review][testing master PR build][discard] comment change

2020-07-23 Thread GitBox
SparkQA removed a comment on pull request #29190: URL: https://github.com/apache/spark/pull/29190#issuecomment-662831076 **[Test build #126387 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126387/testReport)** for PR 29190 at commit [`fa41817`](https://gi

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29166: [SPARK-32280][SPARK-32372][SQL] ResolveReferences.dedupRight should only rewrite attributes for ancestor nodes of the conflict

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29166: [SPARK-32280][SPARK-32372][SQL] ResolveReferences.dedupRight should only rewrite attributes for ancestor nodes of the conflict

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29166: URL: https://github.com/apache/spark/pull/29166#issuecomment-662851587 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins commented on pull request #29192: [SPARK-32393][SQL] Fix postgres bpchar array support

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

[GitHub] [spark] AmplabJenkins commented on pull request #29166: [SPARK-32280][SPARK-32372][SQL] ResolveReferences.dedupRight should only rewrite attributes for ancestor nodes of the conflict plan

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29192: [SPARK-32393][SQL] Fix postgres bpchar array support

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29107: [SPARK-32308][SQL] Move by-name resolution logic of unionByName from API code to analysis phase

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29107: URL: https://github.com/apache/spark/pull/29107#issuecomment-662851223 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662851359 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins commented on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29191: URL: https://github.com/apache/spark/pull/29191#issuecomment-662851513 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 #29014: [SPARK-32199][SPARK-32198] Reduce job failures during decommissioning

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

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

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

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29085: URL: https://github.com/apache/spark/pull/29085#issuecomment-662851559 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins commented on pull request #29186: [SPARK-32386][SS][TESTS] Fix temp view leaking in Structured Streaming tests

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29199: URL: https://github.com/apache/spark/pull/29199#issuecomment-662851429 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29190: [do not review][testing master PR build][discard] comment change

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29186: [SPARK-32386][SS][TESTS] Fix temp view leaking in Structured Streaming tests

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

[GitHub] [spark] AmplabJenkins commented on pull request #29014: [SPARK-32199][SPARK-32198] Reduce job failures during decommissioning

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29191: [SPARK-32364][SQL][FOLLOWUP] Add toMap to return originalMap and documentation

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29191: URL: https://github.com/apache/spark/pull/29191#issuecomment-662851642 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29014: [SPARK-32199][SPARK-32198] Reduce job failures during decommissioning

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29014: URL: https://github.com/apache/spark/pull/29014#issuecomment-662851938 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29190: [do not review][testing master PR build][discard] comment change

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29190: URL: https://github.com/apache/spark/pull/29190#issuecomment-662851556 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29167: URL: https://github.com/apache/spark/pull/29167#issuecomment-662851583 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29186: [SPARK-32386][SS][TESTS] Fix temp view leaking in Structured Streaming tests

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29186: URL: https://github.com/apache/spark/pull/29186#issuecomment-662851912 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29192: [SPARK-32393][SQL] Fix postgres bpchar array support

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29192: URL: https://github.com/apache/spark/pull/29192#issuecomment-662851792 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126

[GitHub] [spark] peter-toth commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

2020-07-23 Thread GitBox
peter-toth commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459253058 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/util/ReuseMap.scala ## @@ -0,0 +1,73 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] [spark] HyukjinKwon removed a comment on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
HyukjinKwon removed a comment on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662844528 I double checked. I believe it should be ready for a look and possibly good to go. This is an autom

[GitHub] [spark] maropu commented on a change in pull request #29200: [SPARK-32251][SQL][TESTS][FOLLOWUP] improve SQL keyword test

2020-07-23 Thread GitBox
maropu commented on a change in pull request #29200: URL: https://github.com/apache/spark/pull/29200#discussion_r459253430 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/SQLKeywordSuite.scala ## @@ -146,11 +159,32 @@ class SQLKeywordSuite extends Spark

[GitHub] [spark] maropu commented on a change in pull request #29200: [SPARK-32251][SQL][TESTS][FOLLOWUP] improve SQL keyword test

2020-07-23 Thread GitBox
maropu commented on a change in pull request #29200: URL: https://github.com/apache/spark/pull/29200#discussion_r459253769 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/SQLKeywordSuite.scala ## @@ -146,11 +159,32 @@ class SQLKeywordSuite extends Spark

[GitHub] [spark] SparkQA commented on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

2020-07-23 Thread GitBox
SparkQA commented on pull request #29199: URL: https://github.com/apache/spark/pull/29199#issuecomment-662852791 **[Test build #126393 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126393/testReport)** for PR 29199 at commit [`01bf8fb`](https://github.com

[GitHub] [spark] AmplabJenkins commented on pull request #29199: [SPARK-32403][SQL] ScriptTransformExec extract common method from process row to avoid repeated judgement

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29199: [SPARK-32403][SQL] Refactor current ScriptTransformationExec

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

[GitHub] [spark] AngersZhuuuu commented on a change in pull request #29199: [SPARK-32403][SQL] Refactor current ScriptTransformationExec

2020-07-23 Thread GitBox
AngersZh commented on a change in pull request #29199: URL: https://github.com/apache/spark/pull/29199#discussion_r459255036 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ## @@ -1063,6 +1063,9 @@ private[hive] trait HiveInspectors {

[GitHub] [spark] peter-toth commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

2020-07-23 Thread GitBox
peter-toth commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459255589 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/Exchange.scala ## @@ -95,46 +89,3 @@ case class ReusedExchangeExec(overr

[GitHub] [spark] peter-toth commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

2020-07-23 Thread GitBox
peter-toth commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459255589 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/Exchange.scala ## @@ -95,46 +89,3 @@ case class ReusedExchangeExec(overr

[GitHub] [spark] imback82 commented on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

2020-07-23 Thread GitBox
imback82 commented on pull request #29167: URL: https://github.com/apache/spark/pull/29167#issuecomment-662856094 retest this please This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [spark] SparkQA commented on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

2020-07-23 Thread GitBox
SparkQA commented on pull request #29167: URL: https://github.com/apache/spark/pull/29167#issuecomment-662858268 **[Test build #126395 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126395/testReport)** for PR 29167 at commit [`477b696`](https://github.com

[GitHub] [spark] AmplabJenkins commented on pull request #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

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

[GitHub] [spark] AmplabJenkins commented on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
AmplabJenkins removed a comment on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662858674 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 #29167: [SPARK-32374][SQL] Disallow setting properties when creating temporary views

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

[GitHub] [spark] LantaoJin commented on pull request #28901: [SPARK-32064][SQL] Supporting create temporary table

2020-07-23 Thread GitBox
LantaoJin commented on pull request #28901: URL: https://github.com/apache/spark/pull/28901#issuecomment-662859641 > how about to use `CACHE TABLE` command to do that? I think you mean `CACHE VIEW` since `CACHE TABLE` still needs to `DROP TABLE` manually. There are four reasons to

[GitHub] [spark] SparkQA commented on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
SparkQA commented on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662858236 **[Test build #126394 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126394/testReport)** for PR 29188 at commit [`a799f91`](https://github.com

[GitHub] [spark] SparkQA commented on pull request #29188: [SPARK-32179][SPARK-32188][PYTHON][DOCS] Replace and redesign the documentation base

2020-07-23 Thread GitBox
SparkQA commented on pull request #29188: URL: https://github.com/apache/spark/pull/29188#issuecomment-662860952 **[Test build #126396 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126396/testReport)** for PR 29188 at commit [`4b918db`](https://github.com

[GitHub] [spark] AmplabJenkins commented on pull request #28911: [SPARK-32077][CORE] Support host-local shuffle data reading when external shuffle service is disabled

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28911: [SPARK-32077][CORE] Support host-local shuffle data reading when external shuffle service is disabled

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

[GitHub] [spark] Ngone51 commented on a change in pull request #28911: [SPARK-32077][CORE] Support host-local shuffle data reading when external shuffle service is disabled

2020-07-23 Thread GitBox
Ngone51 commented on a change in pull request #28911: URL: https://github.com/apache/spark/pull/28911#discussion_r459264839 ## File path: core/src/main/scala/org/apache/spark/internal/config/package.scala ## @@ -1415,10 +1415,16 @@ package object config { private[spark] va

[GitHub] [spark] gengliangwang commented on a change in pull request #28977: [SPARK-32389][TESTS] Add all hive.execution suite in the parallel test group

2020-07-23 Thread GitBox
gengliangwang commented on a change in pull request #28977: URL: https://github.com/apache/spark/pull/28977#discussion_r459266844 ## File path: project/SparkBuild.scala ## @@ -486,9 +485,11 @@ object SparkParallelTestGrouping { ) private val DEFAULT_TEST_GROUP = "defaul

[GitHub] [spark] SparkQA commented on pull request #28911: [SPARK-32077][CORE] Support host-local shuffle data reading when external shuffle service is disabled

2020-07-23 Thread GitBox
SparkQA commented on pull request #28911: URL: https://github.com/apache/spark/pull/28911#issuecomment-662863820 **[Test build #126397 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126397/testReport)** for PR 28911 at commit [`6449229`](https://github.com

[GitHub] [spark] wankunde commented on pull request #29197: [SPARK-32395][SQL] Commit output files after task attempt succeed in dynamic partition datawriter

2020-07-23 Thread GitBox
wankunde commented on pull request #29197: URL: https://github.com/apache/spark/pull/29197#issuecomment-662863980 Check failed because `java.lang.RuntimeException: spark-core: Binary compatibility check failed!` any guys help me?

[GitHub] [spark] peter-toth commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

2020-07-23 Thread GitBox
peter-toth commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459266941 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/util/ReuseMap.scala ## @@ -0,0 +1,73 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] [spark] gengliangwang commented on a change in pull request #28977: [SPARK-32389][TESTS] Add all hive.execution suite in the parallel test group

2020-07-23 Thread GitBox
gengliangwang commented on a change in pull request #28977: URL: https://github.com/apache/spark/pull/28977#discussion_r459267164 ## File path: project/SparkBuild.scala ## @@ -486,9 +485,11 @@ object SparkParallelTestGrouping { ) private val DEFAULT_TEST_GROUP = "defaul

[GitHub] [spark] SparkQA commented on pull request #29168: [SPARK-32375][SQL] Basic functionality of table catalog v2 for JDBC

2020-07-23 Thread GitBox
SparkQA commented on pull request #29168: URL: https://github.com/apache/spark/pull/29168#issuecomment-662866695 **[Test build #126398 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126398/testReport)** for PR 29168 at commit [`56b3329`](https://github.com

[GitHub] [spark] LantaoJin opened a new pull request #29201: [SPARK-32237][SQL][3.0] Resolve hint in CTE

2020-07-23 Thread GitBox
LantaoJin opened a new pull request #29201: URL: https://github.com/apache/spark/pull/29201 ### What changes were proposed in this pull request? The backport of #29062 This PR is to move `Substitution` rule before `Hints` rule in `Analyzer` to avoid hint in CTE not working.

[GitHub] [spark] AmplabJenkins commented on pull request #29168: [SPARK-32375][SQL] Basic functionality of table catalog v2 for JDBC

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29168: [SPARK-32375][SQL] Basic functionality of table catalog v2 for JDBC

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

[GitHub] [spark] peter-toth commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

2020-07-23 Thread GitBox
peter-toth commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459273107 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ReuseExchangeAndSubquerySuite.scala ## @@ -0,0 +1,110 @@ +/* + * Licensed to the

[GitHub] [spark] SparkQA commented on pull request #29201: [SPARK-32237][SQL][3.0] Resolve hint in CTE

2020-07-23 Thread GitBox
SparkQA commented on pull request #29201: URL: https://github.com/apache/spark/pull/29201#issuecomment-662869620 **[Test build #126399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/126399/testReport)** for PR 29201 at commit [`b5be044`](https://github.com

[GitHub] [spark] AmplabJenkins commented on pull request #29201: [SPARK-32237][SQL][3.0] Resolve hint in CTE

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29201: [SPARK-32237][SQL][3.0] Resolve hint in CTE

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

[GitHub] [spark] cloud-fan commented on pull request #29152: [SPARK-32356][SQL] Forbid create view with null type

2020-07-23 Thread GitBox
cloud-fan commented on pull request #29152: URL: https://github.com/apache/spark/pull/29152#issuecomment-662871295 If it just improves error message, I think we don't need to backport. My question is if we want to support it. It seems it's easy to support it, and hive supports it as well.

  1   2   3   4   5   6   7   8   9   10   >