[GitHub] [spark] HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560192539 retest this please This is an automated message from the Apache Git

[GitHub] [spark] HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560095126 retest this please This is an automated message from the

[GitHub] [spark] SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560201356 **[Test build #114689 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114689/testReport)** for PR 26725

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560201742 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics

SparkQA commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics URL: https://github.com/apache/spark/pull/26596#issuecomment-560201633 **[Test build #114697 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114697/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560201737 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] yaooqinn commented on issue #26412: [SPARK-29774][SQL] Date and Timestamp type +/- null should be null as Postgres

yaooqinn commented on issue #26412: [SPARK-29774][SQL] Date and Timestamp type +/- null should be null as Postgres URL: https://github.com/apache/spark/pull/26412#issuecomment-560201716 IIUC, the `NullIntolerant` is not for this type coercion thing.

[GitHub] [spark] SparkQA removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

SparkQA removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560177718 **[Test build #114689 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114689/testReport)** for PR

[GitHub] [spark] amanomer commented on issue #26712: [SPARK-29883][SQL] Improve error messages when function name is an alias

amanomer commented on issue #26712: [SPARK-29883][SQL] Improve error messages when function name is an alias URL: https://github.com/apache/spark/pull/26712#issuecomment-560209320 @gatorsmile Kindly review this PR. This is

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26730: [SPARK-30091][SQL][Python] Document mergeSchema option directly in the Python API

HyukjinKwon commented on a change in pull request #26730: [SPARK-30091][SQL][Python] Document mergeSchema option directly in the Python API URL: https://github.com/apache/spark/pull/26730#discussion_r352405238 ## File path: python/pyspark/sql/readwriter.py ## @@ -300,18

[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

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_r352405260 ## File path:

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26730: [SPARK-30091][SQL][Python] Document mergeSchema option directly in the Python API

HyukjinKwon commented on a change in pull request #26730: [SPARK-30091][SQL][Python] Document mergeSchema option directly in the Python API URL: https://github.com/apache/spark/pull/26730#discussion_r352405238 ## File path: python/pyspark/sql/readwriter.py ## @@ -300,18

[GitHub] [spark] xy2953396112 commented on a change in pull request #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

xy2953396112 commented on a change in pull request #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#discussion_r352405272 ## File path:

[GitHub] [spark] amanomer commented on issue #26731: [SPARK-30093][SQL] Improve error message for creating view

amanomer commented on issue #26731: [SPARK-30093][SQL] Improve error message for creating view URL: https://github.com/apache/spark/pull/26731#issuecomment-560209128 cc @gatorsmile This is an automated message from the

[GitHub] [spark] maropu commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

maropu commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-560213592 ok, I'll try to make a pr later in that approach. Thanks! This is an

[GitHub] [spark] zhengruifeng commented on issue #26547: [SPARK-29914][ML] ML models attach metadata in `transform`/`transformSchema`

zhengruifeng commented on issue #26547: [SPARK-29914][ML] ML models attach metadata in `transform`/`transformSchema` URL: https://github.com/apache/spark/pull/26547#issuecomment-560213404 @srowen Thanks for reviewing! @viirya I had updated this PR according to your comments, could you

[GitHub] [spark] cloud-fan commented on a change in pull request #26712: [SPARK-29883][SQL] Improve error messages when function name is an alias

cloud-fan commented on a change in pull request #26712: [SPARK-29883][SQL] Improve error messages when function name is an alias URL: https://github.com/apache/spark/pull/26712#discussion_r352410699 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13

AmplabJenkins removed a comment on issue #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13 URL: https://github.com/apache/spark/pull/26728#issuecomment-560216580 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13

AmplabJenkins removed a comment on issue #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13 URL: https://github.com/apache/spark/pull/26728#issuecomment-560216586 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] cloud-fan commented on issue #26700: [SPARK-30065][SQL] DataFrameNaFunctions.drop should handle duplicate columns

cloud-fan commented on issue #26700: [SPARK-30065][SQL] DataFrameNaFunctions.drop should handle duplicate columns URL: https://github.com/apache/spark/pull/26700#issuecomment-560223716 thanks, merging to master! This is an

[GitHub] [spark] cloud-fan closed pull request #26700: [SPARK-30065][SQL] DataFrameNaFunctions.drop should handle duplicate columns

cloud-fan closed pull request #26700: [SPARK-30065][SQL] DataFrameNaFunctions.drop should handle duplicate columns URL: https://github.com/apache/spark/pull/26700 This is an automated message from the Apache Git Service. To

[GitHub] [spark] HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560226850 retest this please This is an automated message from the Apache Git

[GitHub] [spark] HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560192539 retest this please This is an automated message from the

[GitHub] [spark] HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560208195 retest this please This is an automated message from the

[GitHub] [spark] SparkQA commented on issue #26680: [SPARK-30047][SQL] Support interval types in UnsafeRow

SparkQA commented on issue #26680: [SPARK-30047][SQL] Support interval types in UnsafeRow URL: https://github.com/apache/spark/pull/26680#issuecomment-560249046 **[Test build #114708 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114708/testReport)** for

[GitHub] [spark] AmplabJenkins removed a comment on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

AmplabJenkins removed a comment on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560164125 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

AmplabJenkins commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560164125 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

AmplabJenkins commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560164129 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

AmplabJenkins removed a comment on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560164129 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560177891 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560177891 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560177890 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560177890 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

AmplabJenkins commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180823 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins removed a comment on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

AmplabJenkins removed a comment on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180816 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

AmplabJenkins commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180816 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

AmplabJenkins removed a comment on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180823 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] SparkQA commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

SparkQA commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180634 **[Test build #114690 has

[GitHub] [spark] LantaoJin commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN

LantaoJin commented on issue #26711: [SPARK-30069][CORE][YARN] Clean up non-shuffle disk block manager files following executor exists on YARN URL: https://github.com/apache/spark/pull/26711#issuecomment-560180523 retest this please

[GitHub] [spark] srowen commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

srowen commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560191791 Hm, that value isn't negative though, just very small. The next line, perhaps accidentally,

[GitHub] [spark] HyukjinKwon commented on issue #26730: [SQL][Python] Document mergeSchema option directly in the Python API

HyukjinKwon commented on issue #26730: [SQL][Python] Document mergeSchema option directly in the Python API URL: https://github.com/apache/spark/pull/26730#issuecomment-560192472 Seems fine but mind filing a JIRA please @nchammas?

[GitHub] [spark] SparkQA commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

SparkQA commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560217473 **[Test build #114701 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

AmplabJenkins removed a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560217699 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

AmplabJenkins removed a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560217701 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

AmplabJenkins commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560217699 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

AmplabJenkins commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560217701 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] cloud-fan commented on issue #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled

cloud-fan commented on issue #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-560225902 thanks, merging to master!

[GitHub] [spark] cloud-fan closed pull request #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled

cloud-fan closed pull request #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled URL: https://github.com/apache/spark/pull/26664 This is an automated message

[GitHub] [spark] HeartSaVioR opened a new pull request #26732: [MINOR][SS] Add implementation note on overriding serialize/deserialize in HDFSMetadataLog methods' scaladoc

HeartSaVioR opened a new pull request #26732: [MINOR][SS] Add implementation note on overriding serialize/deserialize in HDFSMetadataLog methods' scaladoc URL: https://github.com/apache/spark/pull/26732 ### What changes were proposed in this pull request? The patch adds scaladoc on

[GitHub] [spark] xuanyuanking commented on issue #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled

xuanyuanking commented on issue #26664: [SPARK-30074][SQL] The maxNumPostShufflePartitions config should obey reducePostShufflePartitions enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-560229536 Thanks!

[GitHub] [spark] cloud-fan commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

cloud-fan commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560235839 This makes sense. Can we check the behavior of spark 2.4? If it's a behavior

[GitHub] [spark] yaooqinn edited a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

yaooqinn edited a comment on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560240101 In v2.4.4, +(date/timestamp/array/map) are valid, we should add a migration

[GitHub] [spark] cloud-fan commented on a change in pull request #26127: [SPARK-29348][SQL] Add observable Metrics for Streaming queries

cloud-fan commented on a change in pull request #26127: [SPARK-29348][SQL] Add observable Metrics for Streaming queries URL: https://github.com/apache/spark/pull/26127#discussion_r352427417 ## File path:

[GitHub] [spark] cloud-fan closed pull request #26685: [SPARK-30050][SQL] analyze table and rename table should not erase hive table bucketing info

cloud-fan closed pull request #26685: [SPARK-30050][SQL] analyze table and rename table should not erase hive table bucketing info URL: https://github.com/apache/spark/pull/26685 This is an automated message from the Apache

[GitHub] [spark] cloud-fan commented on a change in pull request #26195: [SPARK-29537][SQL] throw exception when user defined a wrong base path

cloud-fan commented on a change in pull request #26195: [SPARK-29537][SQL] throw exception when user defined a wrong base path URL: https://github.com/apache/spark/pull/26195#discussion_r352429554 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243418 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243414 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243418 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243414 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] SparkQA commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking

SparkQA commented on issue #26716: [SPARK-30083][SQL] visitArithmeticUnary should wrap PLUS case with UnaryPositive for type checking URL: https://github.com/apache/spark/pull/26716#issuecomment-560247051 **[Test build #114707 has

[GitHub] [spark] viirya commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large

viirya commented on issue #26722: [SPARK-24666][ML] Fix infinity vectors produced by Word2Vec when numIterations are large URL: https://github.com/apache/spark/pull/26722#issuecomment-560247040 > What I don't really understand is why it would be 'triggered' by the number of partitions

[GitHub] [spark] SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560182562 **[Test build #114692 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114692/testReport)** for PR 26725 at

[GitHub] [spark] dilipbiswal commented on issue #26721: [SPARK-30085][SQL][DOC] Standardize sql reference

dilipbiswal commented on issue #26721: [SPARK-30085][SQL][DOC] Standardize sql reference URL: https://github.com/apache/spark/pull/26721#issuecomment-560182587 LGTM This is an automated message from the Apache Git Service.

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560182714 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560182708 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13

HyukjinKwon commented on a change in pull request #26728: [SPARK-30012][CORE][SQL] Change classes extending scala collection classes to work with 2.13 URL: https://github.com/apache/spark/pull/26728#discussion_r352387387 ## File path:

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark

HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark URL: https://github.com/apache/spark/pull/26496#discussion_r352393084 ## File path: python/pyspark/sql/types.py ## @@ -1463,32 +1474,43 @@ class

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark

HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark URL: https://github.com/apache/spark/pull/26496#discussion_r352393084 ## File path: python/pyspark/sql/types.py ## @@ -1463,32 +1474,43 @@ class

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark

HyukjinKwon commented on a change in pull request #26496: [WIP][SPARK-29748][PYTHON][SQL] Remove Row field sorting in PySpark URL: https://github.com/apache/spark/pull/26496#discussion_r352393084 ## File path: python/pyspark/sql/types.py ## @@ -1463,32 +1474,43 @@ class

[GitHub] [spark] xy2953396112 commented on a change in pull request #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

xy2953396112 commented on a change in pull request #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#discussion_r352402911 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26731: [SPARK-30093][SQL] Improve error message for creating view

AmplabJenkins removed a comment on issue #26731: [SPARK-30093][SQL] Improve error message for creating view URL: https://github.com/apache/spark/pull/26731#issuecomment-560208099 Can one of the admins verify this patch? This

[GitHub] [spark] AmplabJenkins commented on issue #26731: [SPARK-30093][SQL] Improve error message for creating view

AmplabJenkins commented on issue #26731: [SPARK-30093][SQL] Improve error message for creating view URL: https://github.com/apache/spark/pull/26731#issuecomment-560209388 Can one of the admins verify this patch? This is an

[GitHub] [spark] SparkQA commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

SparkQA commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#issuecomment-560209516 **[Test build #114700 has

[GitHub] [spark] SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

SparkQA commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209523 **[Test build #114699 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114699/testReport)** for PR 26725 at

[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

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_r352405641 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209837 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209830 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

AmplabJenkins commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#issuecomment-560209899 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

AmplabJenkins removed a comment on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#issuecomment-560209894 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

AmplabJenkins removed a comment on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#issuecomment-560209899 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize

AmplabJenkins commented on issue #26693: [MINOR][SS] Note that writer should not be closed in RateStreamMicroBatchStream.serialize URL: https://github.com/apache/spark/pull/26693#issuecomment-560209894 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209837 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209830 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560212410 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560212402 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics

AmplabJenkins commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics URL: https://github.com/apache/spark/pull/26596#issuecomment-560215096 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics

AmplabJenkins commented on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics URL: https://github.com/apache/spark/pull/26596#issuecomment-560215090 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics

AmplabJenkins removed a comment on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics URL: https://github.com/apache/spark/pull/26596#issuecomment-560215096 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics

AmplabJenkins removed a comment on issue #26596: [SPARK-29959][ML][PYSPARK] Summarizer support more metrics URL: https://github.com/apache/spark/pull/26596#issuecomment-560215090 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #26729: [SPARK-29851][SQL][FOLLOW-UP] Use foreach instead of misusing map

AmplabJenkins removed a comment on issue #26729: [SPARK-29851][SQL][FOLLOW-UP] Use foreach instead of misusing map URL: https://github.com/apache/spark/pull/26729#issuecomment-560224427 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560224362 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560224359 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26729: [SPARK-29851][SQL][FOLLOW-UP] Use foreach instead of misusing map

AmplabJenkins removed a comment on issue #26729: [SPARK-29851][SQL][FOLLOW-UP] Use foreach instead of misusing map URL: https://github.com/apache/spark/pull/26729#issuecomment-560224424 Merged build finished. Test PASSed.

[GitHub] [spark] cloud-fan commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

cloud-fan commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667#issuecomment-560233147 LGTM except the comments from @viirya This is

[GitHub] [spark] cloud-fan commented on issue #26684: [WIP][SPARK-30001][SQL] ResolveRelations should handle both V1 and V2 tables.

cloud-fan commented on issue #26684: [WIP][SPARK-30001][SQL] ResolveRelations should handle both V1 and V2 tables. URL: https://github.com/apache/spark/pull/26684#issuecomment-560234214 > The current implementation of V2SessionCatalog always returns V1Table. Should we update it to handle

[GitHub] [spark] HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

HyukjinKwon commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243225 retest this please This is an automated message from the Apache Git

[GitHub] [spark] SparkQA removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

SparkQA removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560209523 **[Test build #114699 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114699/testReport)** for PR

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243198 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins removed a comment on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243208 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder

AmplabJenkins commented on issue #26725: [DO-NOT-MERGE] Health Checking in PR Builder URL: https://github.com/apache/spark/pull/26725#issuecomment-560243208 Test PASSed. Refer to this link for build results (access rights to CI server needed):

<    1   2   3   4   5   6   >