[GitHub] [spark] cloud-fan closed pull request #26318: [SPARK-29652][SQL] AlterTable/DescribeTable should not list the table as a child

2019-10-30 Thread GitBox
cloud-fan closed pull request #26318: [SPARK-29652][SQL] AlterTable/DescribeTable should not list the table as a child URL: https://github.com/apache/spark/pull/26318 This is an automated message from the Apache Git Service.

[GitHub] [spark] gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-547999191 I think the proposal is reasonable. After https://github.com/apache/spark/pull/26107, we can't store string type into

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26317: [SPARK-29628][SQL] Forcibly create a temporary view in CREATE VIEW if referencing a temporary view

2019-10-30 Thread GitBox
dongjoon-hyun commented on a change in pull request #26317: [SPARK-29628][SQL] Forcibly create a temporary view in CREATE VIEW if referencing a temporary view URL: https://github.com/apache/spark/pull/26317#discussion_r340731270 ## File path: sql/core/src/main/scala/org/apache/spar

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26319: [SPARK-29594][SQL] Create a Dataset from a Sequence of Case class where…

2019-10-30 Thread GitBox
dongjoon-hyun commented on a change in pull request #26319: [SPARK-29594][SQL] Create a Dataset from a Sequence of Case class where… URL: https://github.com/apache/spark/pull/26319#discussion_r340734036 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaRe

[GitHub] [spark] gengliangwang edited a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
gengliangwang edited a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-547999191 I think the proposal is reasonable. If no values are provided, then handling it as null type makes sense. After

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548004695 Merged build finished. Test PASSed. This is an automated message from the Apache

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548004712 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPu

[GitHub] [spark] mstill3 commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
mstill3 commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340737390 ## File path: python/pyspark/sql/session.py ## @@ -94,12 +94,12 @@ def config(self, key=None, value=None, conf=None

[GitHub] [spark] yaooqinn opened a new pull request #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
yaooqinn opened a new pull request #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325 ### What changes were proposed in this pull request? sum support interval values ### Why are the changes needed?

[GitHub] [spark] cloud-fan commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
cloud-fan commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#discussion_r340738720 ## File path: sql/core/src/test/scala/org/apache/spar

[GitHub] [spark] cloud-fan commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
cloud-fan commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#discussion_r340738564 ## File path: sql/core/src/test/scala/org/apache/spar

[GitHub] [spark] mstill3 commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
mstill3 commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340739262 ## File path: python/pyspark/sql/session.py ## @@ -624,20 +624,20 @@ def createDataFrame(self, data, schema=None,

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548004712 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//jo

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548004695 Merged build finished. Test PASSed. This is an automated message from t

[GitHub] [spark] SparkQA commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
SparkQA commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325#issuecomment-548007913 **[Test build #112937 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112937/testReport)** for

[GitHub] [spark] SparkQA commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548007958 **[Test build #112938 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112938/testReport)** for PR 26299 at commit [`c

[GitHub] [spark] SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548007971 **[Test build #112939 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112939/testReport)** f

[GitHub] [spark] amanomer commented on a change in pull request #26317: [SPARK-29628][SQL] Forcibly create a temporary view in CREATE VIEW if referencing a temporary view

2019-10-30 Thread GitBox
amanomer commented on a change in pull request #26317: [SPARK-29628][SQL] Forcibly create a temporary view in CREATE VIEW if referencing a temporary view URL: https://github.com/apache/spark/pull/26317#discussion_r340741077 ## File path: sql/core/src/main/scala/org/apache/spark/sql

[GitHub] [spark] AmplabJenkins removed a comment on issue #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325#issuecomment-548008763 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.b

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548008598 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPul

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548008833 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkele

[GitHub] [spark] AmplabJenkins removed a comment on issue #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325#issuecomment-548008753 Merged build finished. Test PASSed. This is an a

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548008825 Merged build finished. Test PASSed. This is an automat

[GitHub] [spark] AmplabJenkins commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325#issuecomment-548008753 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] mstill3 commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
mstill3 commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340741298 ## File path: python/pyspark/sql/context.py ## @@ -318,7 +318,7 @@ def registerDataFrameAsTable(self, df, tableName

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548008585 Merged build finished. Test FAILed. This is an automated message from t

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548008825 Merged build finished. Test PASSed. This is an

[GitHub] [spark] planga82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
planga82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#discussion_r340741367 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder

[GitHub] [spark] mstill3 commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
mstill3 commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340741159 ## File path: python/pyspark/sql/dataframe.py ## @@ -1077,7 +1077,7 @@ def sort(self, *cols, **kwargs): ""

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548008585 Merged build finished. Test FAILed. This is an automated message from the Apache

[GitHub] [spark] SparkQA commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548008561 **[Test build #112938 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112938/testReport)** for PR 26299 at commit [`

[GitHub] [spark] AmplabJenkins commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26325: [SPARK-29663][SQL] Support sum with interval type values URL: https://github.com/apache/spark/pull/26325#issuecomment-548008763 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.

[GitHub] [spark] SparkQA removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548007958 **[Test build #112938 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112938/testReport)** for PR 26299 at com

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548008833 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548008598 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job

[GitHub] [spark] squito commented on issue #26272: [SPARK-27736][Core][SHUFFLE] Improve handling of FetchFailures caused by ExternalShuffleService losing track of executor registrations

2019-10-30 Thread GitBox
squito commented on issue #26272: [SPARK-27736][Core][SHUFFLE] Improve handling of FetchFailures caused by ExternalShuffleService losing track of executor registrations URL: https://github.com/apache/spark/pull/26272#issuecomment-548011798 cc @attilapiros

[GitHub] [spark] srowen commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
srowen commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340744846 ## File path: python/pyspark/sql/session.py ## @@ -94,12 +94,12 @@ def config(self, key=None, value=None, conf=None)

[GitHub] [spark] mstill3 commented on a change in pull request #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
mstill3 commented on a change in pull request #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#discussion_r340745233 ## File path: python/pyspark/sql/session.py ## @@ -94,12 +94,12 @@ def config(self, key=None, value=None, conf=None

[GitHub] [spark] SparkQA commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012098 **[Test build #112940 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112940/testReport)** for PR 26299 at commit [`8

[GitHub] [spark] SparkQA commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
SparkQA commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548012148 **[Test build #112941 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112941/testRep

[GitHub] [spark] SparkQA commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012505 **[Test build #112940 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112940/testReport)** for PR 26299 at commit [`

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012528 Merged build finished. Test FAILed. This is an automated message from the Apache

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012544 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPul

[GitHub] [spark] SparkQA removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
SparkQA removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012098 **[Test build #112940 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112940/testReport)** for PR 26299 at com

[GitHub] [spark] AmplabJenkins commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548012812 Merged build finished. Test PASSed. This is a

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012528 Merged build finished. Test FAILed. This is an automated message from t

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012858 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPu

[GitHub] [spark] AmplabJenkins removed a comment on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548012812 Merged build finished. Test PASSed. T

[GitHub] [spark] AmplabJenkins commented on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012838 Merged build finished. Test PASSed. This is an automated message from the Apache

[GitHub] [spark] imback82 commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
imback82 commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#discussion_r340746147 ## File path: sql/core/src/test/scala/org/apache/spark/

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012544 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job

[GitHub] [spark] AmplabJenkins removed a comment on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548012835 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548012835 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012838 Merged build finished. Test PASSed. This is an automated message from t

[GitHub] [spark] AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26299: Fix typo in example documentation URL: https://github.com/apache/spark/pull/26299#issuecomment-548012858 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//jo

[GitHub] [spark] huaxingao commented on issue #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
huaxingao commented on issue #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26303#issuecomment-548013593 @viirya @imback82 Could you please review? Do I need to follow the implementation in htt

[GitHub] [spark] imback82 commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
imback82 commented on a change in pull request #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#discussion_r340748227 ## File path: sql/core/src/test/scala/org/apache/spark/

[GitHub] [spark] SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548015965 **[Test build #112944 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112944/testReport)** f

[GitHub] [spark] SparkQA commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
SparkQA commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#issuecomment-548016025 **[Test build #112942 has started](https://amplab.cs.berkeley.edu/jenkins/job/Spar

[GitHub] [spark] SparkQA commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
SparkQA commented on issue #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#issuecomment-548016022 **[Test build #112943 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112943/testRep

[GitHub] [spark] AmplabJenkins commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#issuecomment-548016654 Test PASSed. Refer to this link for build results (access rights to CI ser

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548016767 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkele

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548016753 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548016767 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs

[GitHub] [spark] AmplabJenkins removed a comment on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#issuecomment-548016648 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#issuecomment-548016654 Test PASSed. Refer to this link for build results (access rights t

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548016753 Merged build finished. Test PASSed. This is an automat

[GitHub] [spark] AmplabJenkins commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26304: [SPARK-29592][SQL] ALTER TABLE (set partition location) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26304#issuecomment-548016648 Merged build finished. Test PASSed.

[GitHub] [spark] imback82 commented on issue #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
imback82 commented on issue #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26303#issuecomment-548017801 This command doesn't need to follow #26041. +@cloud-fan

[GitHub] [spark] imback82 commented on a change in pull request #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands

2019-10-30 Thread GitBox
imback82 commented on a change in pull request #26303: [SPARK-29643][SQL] ALTER TABLE (DROP PARTITION) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26303#discussion_r340752549 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql

[GitHub] [spark] SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
SparkQA commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548021003 **[Test build #112939 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112939/testReport)**

[GitHub] [spark] SparkQA removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
SparkQA removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548007971 **[Test build #112939 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112939/testRepo

[GitHub] [spark] maryannxue commented on a change in pull request #26289: [SPARK-28560][SQL][followup] support the build side to local shuffle reader as far as possible in BroadcastHashJoin

2019-10-30 Thread GitBox
maryannxue commented on a change in pull request #26289: [SPARK-28560][SQL][followup] support the build side to local shuffle reader as far as possible in BroadcastHashJoin URL: https://github.com/apache/spark/pull/26289#discussion_r340724212 ## File path: sql/core/src/main/scala/

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548021597 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley

[GitHub] [spark] AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548021581 Merged build finished. Test PASSed. This is an automat

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548021581 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26110: [SPARK-29126][PYSPARK][DOC] Pandas Cogroup udf usage guide URL: https://github.com/apache/spark/pull/26110#issuecomment-548021597 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.

[GitHub] [spark] amanomer opened a new pull request #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
amanomer opened a new pull request #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324 ### What changes were proposed in this pull request? During creation of array, if CreateArray does not gets any children to set dat

[GitHub] [spark] cloud-fan commented on a change in pull request #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
cloud-fan commented on a change in pull request #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#discussion_r340757675 ## File path: sql/catalyst/src/main/java/org/apache/spark/sql/connector/expressions/Expressions.java ## @@ -82,8 +82,10 @@ public st

[GitHub] [spark] SparkQA commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
SparkQA commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-548022653 **[Test build #112926 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112926/testReport)** for PR 26290

[GitHub] [spark] cloud-fan commented on a change in pull request #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
cloud-fan commented on a change in pull request #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#discussion_r340758635 ## File path: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ## @@ -258,7 +258,21 @@ final class DataFrameWriter[

[GitHub] [spark] SparkQA commented on issue #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
SparkQA commented on issue #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#issuecomment-548023825 **[Test build #112945 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112945/testReport)** for PR 26297 at commit [`aec3b6b`](https://gi

[GitHub] [spark] AmplabJenkins commented on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548023641 Can one of the admins verify this patch? This is an auto

[GitHub] [spark] SparkQA removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
SparkQA removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-547915761 **[Test build #112926 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/112926/testReport)** for P

[GitHub] [spark] AmplabJenkins commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-548023795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins

[GitHub] [spark] AmplabJenkins commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-548023785 Merged build finished. Test PASSed. This is an automated message f

[GitHub] [spark] AmplabJenkins removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-548023795 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu

[GitHub] [spark] AmplabJenkins commented on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548024383 Can one of the admins verify this patch? This is an auto

[GitHub] [spark] AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-547871737 Can one of the admins verify this patch? This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26290: [SPARK-29120][SQL][TESTS] Port create_view.sql URL: https://github.com/apache/spark/pull/26290#issuecomment-548023785 Merged build finished. Test PASSed. This is an automated m

[GitHub] [spark] AmplabJenkins commented on issue #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#issuecomment-548024579 Merged build finished. Test PASSed. This is an automated message from the Apache Git Service. To re

[GitHub] [spark] AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548023641 Can one of the admins verify this patch? This is

[GitHub] [spark] AmplabJenkins commented on issue #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
AmplabJenkins commented on issue #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#issuecomment-548024585 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8

[GitHub] [spark] AmplabJenkins removed a comment on issue #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#issuecomment-548024579 Merged build finished. Test PASSed. This is an automated message from the Apache Git Servic

[GitHub] [spark] AmplabJenkins removed a comment on issue #26297: [DO NOT MERGE]

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26297: [DO NOT MERGE] URL: https://github.com/apache/spark/pull/26297#issuecomment-548024585 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBu

[GitHub] [spark] gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548025012 ok to test This is an automated message from the Apache

[GitHub] [spark] AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
AmplabJenkins removed a comment on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548024383 Can one of the admins verify this patch? This is

[GitHub] [spark] gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array

2019-10-30 Thread GitBox
gengliangwang commented on issue #26324: [SPARK-29462] The data type of "array()" should be array URL: https://github.com/apache/spark/pull/26324#issuecomment-548025251 @amanomer Could you add some test cases? This is an auto

[GitHub] [spark] imback82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
imback82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#discussion_r340760416 ## File path: sql/core/src/test/scala/org/apache/spark/sql/connector/DataSourceV2SQLSuite

[GitHub] [spark] imback82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution.

2019-10-30 Thread GitBox
imback82 commented on a change in pull request #26182: [SPARK-29523][SQL] SHOW COLUMNS should do multi-catalog resolution. URL: https://github.com/apache/spark/pull/26182#discussion_r340755197 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g

<    3   4   5   6   7   8   9   10   11   >