[GitHub] [spark] AmplabJenkins commented on pull request #28386: [SPARK-31517][R] fix strategy for handling ... names in mutate

AmplabJenkins commented on pull request #28386: URL: https://github.com/apache/spark/pull/28386#issuecomment-620419395 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 #28386: [SPARK-31517][R] fix strategy for handling ... names in mutate

SparkQA commented on pull request #28386: URL: https://github.com/apache/spark/pull/28386#issuecomment-620418942 **[Test build #121964 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121964/testReport)** for PR 28386 at commit [`04e50a9`](https://github.com

[GitHub] [spark] MichaelChirico commented on a change in pull request #28386: [SPARK-31517][R] fix strategy for handling ... names in mutate

MichaelChirico commented on a change in pull request #28386: URL: https://github.com/apache/spark/pull/28386#discussion_r416372474 ## File path: R/pkg/R/DataFrame.R ## @@ -2287,16 +2287,19 @@ setMethod("mutate", # For named arguments, use the names for arguments

[GitHub] [spark] baohe-zhang commented on pull request #28378: [SPARK-31584][WebUI] Fix NullPointerException when parsing event log with InMemoryStore

baohe-zhang commented on pull request #28378: URL: https://github.com/apache/spark/pull/28378#issuecomment-620416506 @gengliangwang sure, I will add unit tests. This is an automated message from the Apache Git Service. To res

[GitHub] [spark] gengliangwang edited a comment on pull request #28378: [SPARK-31584][WebUI] Fix NullPointerException when parsing event log with InMemoryStore

gengliangwang edited a comment on pull request #28378: URL: https://github.com/apache/spark/pull/28378#issuecomment-620409199 @baohe-zhang Thanks for the fix! Could you follow @HeartSaVioR 's suggestion to add unit tests?

[GitHub] [spark] MichaelChirico commented on pull request #28386: [SPARK-31517][R] fix strategy for handling ... names in mutate

MichaelChirico commented on pull request #28386: URL: https://github.com/apache/spark/pull/28386#issuecomment-620416479 @HyukjinKwon please advise on how to test this. I did an ad hoc test that it's working as intended but we should formalize. -

[GitHub] [spark] MichaelChirico opened a new pull request #28386: [SPARK-31517][R] fix strategy for handling ... names in mutate

MichaelChirico opened a new pull request #28386: URL: https://github.com/apache/spark/pull/28386 ### What changes were proposed in this pull request? Change the strategy for how the varargs are handled in the default `mutate` method ### Why are the changes needed?

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28370: [SPARK-20732][CORE] Decommission cache blocks to other executors when an executor is decommissioned

AmplabJenkins removed a comment on pull request #28370: URL: https://github.com/apache/spark/pull/28370#issuecomment-620415137 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28370: [SPARK-20732][CORE] Decommission cache blocks to other executors when an executor is decommissioned

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

[GitHub] [spark] AmplabJenkins commented on pull request #28370: [SPARK-20732][CORE] Decommission cache blocks to other executors when an executor is decommissioned

AmplabJenkins commented on pull request #28370: URL: https://github.com/apache/spark/pull/28370#issuecomment-620415124 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 #28370: [SPARK-20732][CORE] Decommission cache blocks to other executors when an executor is decommissioned

SparkQA removed a comment on pull request #28370: URL: https://github.com/apache/spark/pull/28370#issuecomment-620299487 **[Test build #121934 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121934/testReport)** for PR 28370 at commit [`bb324f9`](https://gi

[GitHub] [spark] SparkQA commented on pull request #28370: [SPARK-20732][CORE] Decommission cache blocks to other executors when an executor is decommissioned

SparkQA commented on pull request #28370: URL: https://github.com/apache/spark/pull/28370#issuecomment-620414158 **[Test build #121934 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121934/testReport)** for PR 28370 at commit [`bb324f9`](https://github.co

[GitHub] [spark] AmplabJenkins removed a comment on pull request #27690: [SPARK-21514][SQL] Added a new option to use non-blobstore storage when writing into blobstore storage

AmplabJenkins removed a comment on pull request #27690: URL: https://github.com/apache/spark/pull/27690#issuecomment-620413472 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #27690: [SPARK-21514][SQL] Added a new option to use non-blobstore storage when writing into blobstore storage

AmplabJenkins commented on pull request #27690: URL: https://github.com/apache/spark/pull/27690#issuecomment-620413472 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 #27690: [SPARK-21514][SQL] Added a new option to use non-blobstore storage when writing into blobstore storage

SparkQA commented on pull request #27690: URL: https://github.com/apache/spark/pull/27690#issuecomment-620412995 **[Test build #121963 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121963/testReport)** for PR 27690 at commit [`4d525f0`](https://github.com

[GitHub] [spark] kiszk commented on pull request #27690: [SPARK-21514][SQL] Added a new option to use non-blobstore storage when writing into blobstore storage

kiszk commented on pull request #27690: URL: https://github.com/apache/spark/pull/27690#issuecomment-620411164 retest this please This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [spark] AmplabJenkins removed a comment on pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

AmplabJenkins removed a comment on pull request #27803: URL: https://github.com/apache/spark/pull/27803#issuecomment-620410579 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

AmplabJenkins commented on pull request #27803: URL: https://github.com/apache/spark/pull/27803#issuecomment-620410579 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 #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

SparkQA commented on pull request #27803: URL: https://github.com/apache/spark/pull/27803#issuecomment-620410205 **[Test build #121962 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121962/testReport)** for PR 27803 at commit [`a131ccf`](https://github.com

[GitHub] [spark] HyukjinKwon commented on pull request #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

HyukjinKwon commented on pull request #28379: URL: https://github.com/apache/spark/pull/28379#issuecomment-620409919 cc also @felixcheung and @shivaram This is an automated message from the Apache Git Service. To respond to

[GitHub] [spark] gengliangwang commented on pull request #28378: [SPARK-31584][WebUI] Fix NullPointerException when parsing event log with InMemoryStore

gengliangwang commented on pull request #28378: URL: https://github.com/apache/spark/pull/28378#issuecomment-620409199 @baohe-zhang Thanks for the fix! Could you follow @HeartSaVioR 's to add unit tests? This is an automa

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28377: [SPARK-27340][SS][2.4] Alias on TimeWindow expression may cause watermark metadata lost

AmplabJenkins removed a comment on pull request #28377: URL: https://github.com/apache/spark/pull/28377#issuecomment-620407764 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #28377: [SPARK-27340][SS][2.4] Alias on TimeWindow expression may cause watermark metadata lost

AmplabJenkins commented on pull request #28377: URL: https://github.com/apache/spark/pull/28377#issuecomment-620407764 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 #28385: [SPARK-31591][CORE] Fix null name prefix when create directory

AmplabJenkins commented on pull request #28385: URL: https://github.com/apache/spark/pull/28385#issuecomment-620407779 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 #28385: [SPARK-31591][CORE] Fix null name prefix when create directory

AmplabJenkins removed a comment on pull request #28385: URL: https://github.com/apache/spark/pull/28385#issuecomment-620407779 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA commented on pull request #28385: [SPARK-31591][CORE] Fix null name prefix when create directory

SparkQA commented on pull request #28385: URL: https://github.com/apache/spark/pull/28385#issuecomment-620407468 **[Test build #121961 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121961/testReport)** for PR 28385 at commit [`a42f005`](https://github.com

[GitHub] [spark] SparkQA removed a comment on pull request #28377: [SPARK-27340][SS][2.4] Alias on TimeWindow expression may cause watermark metadata lost

SparkQA removed a comment on pull request #28377: URL: https://github.com/apache/spark/pull/28377#issuecomment-620306258 **[Test build #121936 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121936/testReport)** for PR 28377 at commit [`b6e410b`](https://gi

[GitHub] [spark] maropu commented on a change in pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

maropu commented on a change in pull request #27803: URL: https://github.com/apache/spark/pull/27803#discussion_r416360187 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisErrorSuite.scala ## @@ -425,38 +425,35 @@ class AnalysisErrorSuit

[GitHub] [spark] SparkQA commented on pull request #28377: [SPARK-27340][SS][2.4] Alias on TimeWindow expression may cause watermark metadata lost

SparkQA commented on pull request #28377: URL: https://github.com/apache/spark/pull/28377#issuecomment-620406736 **[Test build #121936 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121936/testReport)** for PR 28377 at commit [`b6e410b`](https://github.co

[GitHub] [spark] maropu commented on a change in pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

maropu commented on a change in pull request #27803: URL: https://github.com/apache/spark/pull/27803#discussion_r416359123 ## File path: sql/core/src/test/scala/org/apache/spark/sql/GeneratorFunctionSuite.scala ## @@ -344,12 +345,73 @@ class GeneratorFunctionSuite extends Quer

[GitHub] [spark] HeartSaVioR commented on pull request #28378: [SPARK-31584][WebUI] Fix NullPointerException when parsing event log with InMemoryStore

HeartSaVioR commented on pull request #28378: URL: https://github.com/apache/spark/pull/28378#issuecomment-620405834 cc. @gengliangwang This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

AmplabJenkins removed a comment on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620405134 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 #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

AmplabJenkins removed a comment on pull request #28379: URL: https://github.com/apache/spark/pull/28379#issuecomment-620404728 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

AmplabJenkins commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620405134 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [spark] LantaoJin opened a new pull request #28385: [SPARK-31591][CORE] Fix null name prefix when create directory

LantaoJin opened a new pull request #28385: URL: https://github.com/apache/spark/pull/28385 ### What changes were proposed in this pull request? In our production, we find that many shuffle files could be located in /hadoop/2/yarn/local/usercache/b_carmel/appcache/application_15864878

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

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

[GitHub] [spark] yaooqinn commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

yaooqinn commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620404977 cc @cloud-fan, please also notice that the flakiness of CliSuite is still there according the test failure above. -

[GitHub] [spark] dbtsai commented on pull request #28376: [SPARK-31582] [Yarn] Being able to not populate Hadoop classpath

dbtsai commented on pull request #28376: URL: https://github.com/apache/spark/pull/28376#issuecomment-620404585 Ping @jerryshao @tgravescs @ulysses-you who work on this area for more eyes. This is an automated message from th

[GitHub] [spark] SparkQA commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

SparkQA commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620404705 **[Test build #121960 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121960/testReport)** for PR 28381 at commit [`17b0438`](https://github.com

[GitHub] [spark] AmplabJenkins commented on pull request #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

AmplabJenkins commented on pull request #28379: URL: https://github.com/apache/spark/pull/28379#issuecomment-620404718 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 #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

AmplabJenkins removed a comment on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620404013 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

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

[GitHub] [spark] yaooqinn commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

yaooqinn commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620404507 retest this please This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [spark] maropu commented on pull request #28106: [SPARK-31335][SQL] Add try function support

maropu commented on pull request #28106: URL: https://github.com/apache/spark/pull/28106#issuecomment-620403960 Yea, I'm also +1 on this idea. But, we need more feedbacks from others. This is an automated message from the Apa

[GitHub] [spark] AmplabJenkins commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

AmplabJenkins commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620404007 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [spark] maropu edited a comment on pull request #28106: [SPARK-31335][SQL] Add try function support

maropu edited a comment on pull request #28106: URL: https://github.com/apache/spark/pull/28106#issuecomment-620403960 Yea, I'm also +1 on this idea. But, we need more feedbacks from others. also cc: @dongjoon-hyun This is

[GitHub] [spark] SparkQA removed a comment on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

SparkQA removed a comment on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620344715 **[Test build #121946 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121946/testReport)** for PR 28381 at commit [`17b0438`](https://gi

[GitHub] [spark] SparkQA commented on pull request #28381: [SPARK-31586][SQL] Replace expression TimeSub(l, r) with TimeAdd(l -r)

SparkQA commented on pull request #28381: URL: https://github.com/apache/spark/pull/28381#issuecomment-620403432 **[Test build #121946 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121946/testReport)** for PR 28381 at commit [`17b0438`](https://github.co

[GitHub] [spark] SparkQA removed a comment on pull request #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

SparkQA removed a comment on pull request #28379: URL: https://github.com/apache/spark/pull/28379#issuecomment-620340295 **[Test build #121944 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121944/testReport)** for PR 28379 at commit [`27f15ab`](https://gi

[GitHub] [spark] SparkQA commented on pull request #28379: [SPARK-28040][SPARK-28070][R] Write type object s3

SparkQA commented on pull request #28379: URL: https://github.com/apache/spark/pull/28379#issuecomment-620402749 **[Test build #121944 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121944/testReport)** for PR 28379 at commit [`27f15ab`](https://github.co

[GitHub] [spark] MaxGekk commented on a change in pull request #28343: [SPARK-31563][SQL] Fix failure of InSet.sql for collections of Catalyst's internal types

MaxGekk commented on a change in pull request #28343: URL: https://github.com/apache/spark/pull/28343#discussion_r416351119 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +520,9 @@ case class InSet(child: Expre

[GitHub] [spark] MaxGekk commented on a change in pull request #28343: [SPARK-31563][SQL] Fix failure of InSet.sql for collections of Catalyst's internal types

MaxGekk commented on a change in pull request #28343: URL: https://github.com/apache/spark/pull/28343#discussion_r416350651 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +520,9 @@ case class InSet(child: Expre

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

AmplabJenkins removed a comment on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620399220 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

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

[GitHub] [spark] imback82 commented on a change in pull request #28375: [SPARK-30282][SQL][FOLLOWUP] SHOW TBLPROPERTIES should support views

imback82 commented on a change in pull request #28375: URL: https://github.com/apache/spark/pull/28375#discussion_r416349712 ## File path: docs/sql-migration-guide.md ## @@ -59,7 +59,7 @@ license: | - In Spark 3.0, you can use `ADD FILE` to add file directories as well.

[GitHub] [spark] AmplabJenkins commented on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

AmplabJenkins commented on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620399217 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [spark] MaxGekk commented on pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

MaxGekk commented on pull request #28328: URL: https://github.com/apache/spark/pull/28328#issuecomment-620399241 After offline discussion with @gatorsmile @cloud-fan @HyukjinKwon, we decided to revert https://github.com/apache/spark/pull/25754 . I will open a PR for that and close this PR.

[GitHub] [spark] imback82 commented on a change in pull request #28375: [SPARK-30282][SQL][FOLLOWUP] SHOW TBLPROPERTIES should support views

imback82 commented on a change in pull request #28375: URL: https://github.com/apache/spark/pull/28375#discussion_r416349786 ## File path: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveCommandSuite.scala ## @@ -193,18 +193,16 @@ class HiveCommandSuite extends

[GitHub] [spark] SparkQA removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

SparkQA removed a comment on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620342487 **[Test build #121945 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121945/testReport)** for PR 28380 at commit [`7203521`](https://gi

[GitHub] [spark] MaxGekk commented on a change in pull request #28343: [SPARK-31563][SQL] Fix failure of InSet.sql for collections of Catalyst's internal types

MaxGekk commented on a change in pull request #28343: URL: https://github.com/apache/spark/pull/28343#discussion_r416348966 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +520,9 @@ case class InSet(child: Expre

[GitHub] [spark] SparkQA commented on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

SparkQA commented on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620398517 **[Test build #121945 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121945/testReport)** for PR 28380 at commit [`7203521`](https://github.co

[GitHub] [spark] cloud-fan commented on a change in pull request #28375: [SPARK-30282][SQL][FOLLOWUP] SHOW TBLPROPERTIES should support views

cloud-fan commented on a change in pull request #28375: URL: https://github.com/apache/spark/pull/28375#discussion_r416344437 ## File path: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveCommandSuite.scala ## @@ -193,18 +193,16 @@ class HiveCommandSuite extend

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28365: [SPARK-31571][R] Overhaul stop/message/warning calls to be more translation-friendly/canonical

AmplabJenkins removed a comment on pull request #28365: URL: https://github.com/apache/spark/pull/28365#issuecomment-620394587 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] cloud-fan commented on a change in pull request #28375: [SPARK-30282][SQL][FOLLOWUP] SHOW TBLPROPERTIES should support views

cloud-fan commented on a change in pull request #28375: URL: https://github.com/apache/spark/pull/28375#discussion_r416343587 ## File path: docs/sql-migration-guide.md ## @@ -59,7 +59,7 @@ license: | - In Spark 3.0, you can use `ADD FILE` to add file directories as well.

[GitHub] [spark] SparkQA removed a comment on pull request #28365: [SPARK-31571][R] Overhaul stop/message/warning calls to be more translation-friendly/canonical

SparkQA removed a comment on pull request #28365: URL: https://github.com/apache/spark/pull/28365#issuecomment-620376774 **[Test build #121956 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121956/testReport)** for PR 28365 at commit [`06fac7d`](https://gi

[GitHub] [spark] AmplabJenkins commented on pull request #28365: [SPARK-31571][R] Overhaul stop/message/warning calls to be more translation-friendly/canonical

AmplabJenkins commented on pull request #28365: URL: https://github.com/apache/spark/pull/28365#issuecomment-620394587 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 #28365: [SPARK-31571][R] Overhaul stop/message/warning calls to be more translation-friendly/canonical

SparkQA commented on pull request #28365: URL: https://github.com/apache/spark/pull/28365#issuecomment-620394336 **[Test build #121956 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121956/testReport)** for PR 28365 at commit [`06fac7d`](https://github.co

[GitHub] [spark] AmplabJenkins removed a comment on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative t

AmplabJenkins removed a comment on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620391106 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] AmplabJenkins commented on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

AmplabJenkins commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620391106 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 #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

SparkQA commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620390795 **[Test build #121959 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121959/testReport)** for PR 26339 at commit [`4e162c5`](https://github.com

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

AmplabJenkins removed a comment on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620389748 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

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

[GitHub] [spark] AmplabJenkins commented on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

AmplabJenkins commented on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620389743 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [spark] sarutak commented on pull request #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

sarutak commented on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620389486 Ah, I see. Thanks. This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [spark] SparkQA commented on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

SparkQA commented on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620389349 **[Test build #121943 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121943/testReport)** for PR 28380 at commit [`5dc15cd`](https://github.co

[GitHub] [spark] SparkQA removed a comment on pull request #28380: [SPARK-31587][R][INFRA] Uses R 4.0.0 in Github Actions

SparkQA removed a comment on pull request #28380: URL: https://github.com/apache/spark/pull/28380#issuecomment-620340283 **[Test build #121943 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121943/testReport)** for PR 28380 at commit [`5dc15cd`](https://gi

[GitHub] [spark] dongjoon-hyun edited a comment on pull request #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

dongjoon-hyun edited a comment on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620389265 Maybe, we need to re-trigger this once more because this will be terminated at PST midnight. This

[GitHub] [spark] SparkQA removed a comment on pull request #28367: [SPARK-31573][R] Apply fixed=TRUE as appropriate to regex usage in R

SparkQA removed a comment on pull request #28367: URL: https://github.com/apache/spark/pull/28367#issuecomment-620376433 **[Test build #121955 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121955/testReport)** for PR 28367 at commit [`cb6d263`](https://gi

[GitHub] [spark] dongjoon-hyun commented on pull request #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

dongjoon-hyun commented on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620389265 Maybe, we need to retriever this once more because this will be terminated at PST midnight. This is an au

[GitHub] [spark] AmplabJenkins commented on pull request #28367: [SPARK-31573][R] Apply fixed=TRUE as appropriate to regex usage in R

AmplabJenkins commented on pull request #28367: URL: https://github.com/apache/spark/pull/28367#issuecomment-620389185 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 #28367: [SPARK-31573][R] Apply fixed=TRUE as appropriate to regex usage in R

AmplabJenkins removed a comment on pull request #28367: URL: https://github.com/apache/spark/pull/28367#issuecomment-620389185 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] SparkQA commented on pull request #28367: [SPARK-31573][R] Apply fixed=TRUE as appropriate to regex usage in R

SparkQA commented on pull request #28367: URL: https://github.com/apache/spark/pull/28367#issuecomment-620388994 **[Test build #121955 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121955/testReport)** for PR 28367 at commit [`cb6d263`](https://github.co

[GitHub] [spark] AmplabJenkins removed a comment on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative t

AmplabJenkins removed a comment on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620388306 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121

[GitHub] [spark] AmplabJenkins commented on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

AmplabJenkins commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620388303 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 #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative t

AmplabJenkins removed a comment on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620388303 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 #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

SparkQA removed a comment on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620386758 **[Test build #121958 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121958/testReport)** for PR 26339 at commit [`9c4532e`](https://gi

[GitHub] [spark] SparkQA commented on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

SparkQA commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620388294 **[Test build #121958 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121958/testReport)** for PR 26339 at commit [`9c4532e`](https://github.co

[GitHub] [spark] HyukjinKwon commented on pull request #28328: [SPARK-31553][SQL] Fix isInCollection for collection sizes above the optimisation threshold

HyukjinKwon commented on pull request #28328: URL: https://github.com/apache/spark/pull/28328#issuecomment-620387797 Okay, the more I look, the more it makes me to think we should revert #25754 rather than adding bandaid fixes. Shall we revert?

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

dongjoon-hyun commented on a change in pull request #27803: URL: https://github.com/apache/spark/pull/27803#discussion_r416334410 ## File path: sql/core/src/test/scala/org/apache/spark/sql/GeneratorFunctionSuite.scala ## @@ -344,12 +345,73 @@ class GeneratorFunctionSuite exten

[GitHub] [spark] cloud-fan commented on a change in pull request #28343: [SPARK-31563][SQL] Fix failure of InSet.sql for collections of Catalyst's internal types

cloud-fan commented on a change in pull request #28343: URL: https://github.com/apache/spark/pull/28343#discussion_r416334206 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -519,7 +520,9 @@ case class InSet(child: Exp

[GitHub] [spark] AmplabJenkins commented on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

AmplabJenkins commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620387046 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 #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

AmplabJenkins commented on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620386990 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 #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

AmplabJenkins removed a comment on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620386990 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 #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative t

AmplabJenkins removed a comment on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620387046 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] MichaelChirico commented on pull request #28365: [SPARK-31571][R] Overhaul stop/message/warning calls to be more translation-friendly/canonical

MichaelChirico commented on pull request #28365: URL: https://github.com/apache/spark/pull/28365#issuecomment-620386789 Not sure why the Documents task failed, looks like the `roxygen2` part succeeded This is an automated me

[GitHub] [spark] SparkQA commented on pull request #26339: [SPARK-27194][SPARK-29302][SQL] Fix the issue that for dynamic partition overwrite a task would conflict with its speculative task

SparkQA commented on pull request #26339: URL: https://github.com/apache/spark/pull/26339#issuecomment-620386758 **[Test build #121958 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121958/testReport)** for PR 26339 at commit [`9c4532e`](https://github.com

[GitHub] [spark] SparkQA commented on pull request #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

SparkQA commented on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620386738 **[Test build #121957 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121957/testReport)** for PR 28359 at commit [`90d3dbf`](https://github.com

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

dongjoon-hyun commented on a change in pull request #27803: URL: https://github.com/apache/spark/pull/27803#discussion_r416333033 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ## @@ -2203,13 +2220,41 @@ class Analyzer( }

[GitHub] [spark] HeartSaVioR edited a comment on pull request #28326: [SPARK-27340][SS] Alias on TimeWindow expression cause watermark metadata lost

HeartSaVioR edited a comment on pull request #28326: URL: https://github.com/apache/spark/pull/28326#issuecomment-620310834 Sorry for maybe out-of-topic, but I'm seeing that column metadata may have more issues due to not properly defined its usage and how it should be handled. I'm a

[GitHub] [spark] sarutak commented on pull request #28359: [SPARK-31534][WEBUI][3.0] Text for tooltip should be escaped

sarutak commented on pull request #28359: URL: https://github.com/apache/spark/pull/28359#issuecomment-620385316 retest this please. This is an automated message from the Apache Git Service. To respond to the message, please

  1   2   3   4   5   6   7   8   9   10   >