[GitHub] [spark] gatorsmile commented on issue #24618: [SPARK-27734][CORE][SQL] Add memory based thresholds for shuffle spill

2020-02-26 Thread GitBox
gatorsmile commented on issue #24618: [SPARK-27734][CORE][SQL] Add memory based thresholds for shuffle spill URL: https://github.com/apache/spark/pull/24618#issuecomment-591798017 ok to test This is an automated message from

[GitHub] [spark] cloud-fan commented on a change in pull request #27710: [SPARK-30960][SQL] add back the legacy date/timestamp format support in CSV/JSON parser

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27710: [SPARK-30960][SQL] add back the legacy date/timestamp format support in CSV/JSON parser URL: https://github.com/apache/spark/pull/27710#discussion_r384926610 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catal

[GitHub] [spark] MaxGekk commented on a change in pull request #27710: [SPARK-30960][SQL] add back the legacy date/timestamp format support in CSV/JSON parser

2020-02-26 Thread GitBox
MaxGekk commented on a change in pull request #27710: [SPARK-30960][SQL] add back the legacy date/timestamp format support in CSV/JSON parser URL: https://github.com/apache/spark/pull/27710#discussion_r384923946 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalys

[GitHub] [spark] cloud-fan commented on issue #27699: [SPARK-27924][SQL][FOLLOW-UP] Improve ANSI SQL Boolean-Predicate

2020-02-26 Thread GitBox
cloud-fan commented on issue #27699: [SPARK-27924][SQL][FOLLOW-UP] Improve ANSI SQL Boolean-Predicate URL: https://github.com/apache/spark/pull/27699#issuecomment-591792955 thanks, merging to master/3.0! This is an automated

[GitHub] [spark] cloud-fan closed pull request #27699: [SPARK-27924][SQL][FOLLOW-UP] Improve ANSI SQL Boolean-Predicate

2020-02-26 Thread GitBox
cloud-fan closed pull request #27699: [SPARK-27924][SQL][FOLLOW-UP] Improve ANSI SQL Boolean-Predicate URL: https://github.com/apache/spark/pull/27699 This is an automated message from the Apache Git Service. To respond to t

[GitHub] [spark] cloud-fan commented on a change in pull request #25620: [SPARK-25341][Core] Support rolling back a shuffle map stage and re-generate the shuffle files

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #25620: [SPARK-25341][Core] Support rolling back a shuffle map stage and re-generate the shuffle files URL: https://github.com/apache/spark/pull/25620#discussion_r384922916 ## File path: common/network-shuffle/src/main/java/org/apache

[GitHub] [spark] xuanyuanking commented on issue #27667: [SPARK-28228][SQL][TESTS] Refactoring for nested CTE tests

2020-02-26 Thread GitBox
xuanyuanking commented on issue #27667: [SPARK-28228][SQL][TESTS] Refactoring for nested CTE tests URL: https://github.com/apache/spark/pull/27667#issuecomment-591792447 Thanks for the review! This is an automated message fro

[GitHub] [spark] cloud-fan commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#discussion_r384920729 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ##

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591788440 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591788436 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591788436 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591788440 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591752192 **[Test build #119005 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11900

[GitHub] [spark] SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591788161 **[Test build #119005 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119005/testR

[GitHub] [spark] dongjoon-hyun edited a comment on issue #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
dongjoon-hyun edited a comment on issue #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#issuecomment-591787034 +1, late LGTM. You can ignore the above comment because those are too minor, @iRakson . ---

[GitHub] [spark] dongjoon-hyun commented on issue #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
dongjoon-hyun commented on issue #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#issuecomment-591787034 +1, LGTM. You can ignore the above comment because those are too minor, @iRakson . ---

[GitHub] [spark] iRakson commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
iRakson commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#discussion_r384918063 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/hash.sc

[GitHub] [spark] srowen commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled

2020-02-26 Thread GitBox
srowen commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled URL: https://github.com/apache/spark/pull/23495#discussion_r384916414 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonS

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#discussion_r384915402 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/h

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#discussion_r384915586 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#discussion_r384915364 ## File path: docs/sql-migration-guide.md ## @@ -224,6 +224,8 @@ license: |

[GitHub] [spark] gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled

2020-02-26 Thread GitBox
gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled URL: https://github.com/apache/spark/pull/23495#discussion_r384913935 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/J

[GitHub] [spark] gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled

2020-02-26 Thread GitBox
gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled URL: https://github.com/apache/spark/pull/23495#discussion_r384912956 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/J

[GitHub] [spark] gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled

2020-02-26 Thread GitBox
gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled URL: https://github.com/apache/spark/pull/23495#discussion_r384912551 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/J

[GitHub] [spark] gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled

2020-02-26 Thread GitBox
gatorsmile commented on a change in pull request #23495: [SPARK-26503][CORE] Get rid of spark.sql.legacy.timeParser.enabled URL: https://github.com/apache/spark/pull/23495#discussion_r384910825 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/J

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591777152 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591777140 Test FAILed. Refer to this link for build results (access rights to CI server needed): http

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591777150 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591777150 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591777152 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384910262 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst

[GitHub] [spark] AmplabJenkins removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591777134 Merged build finished. Test FAILed. -

[GitHub] [spark] imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384910146 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst

[GitHub] [spark] AmplabJenkins commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591777140 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] AmplabJenkins commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591777134 Merged build finished. Test FAILed. This

[GitHub] [spark] SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591729213 **[Test build #118999 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11899

[GitHub] [spark] SparkQA removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591729214 **[Test build #119000 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1

[GitHub] [spark] SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591776629 **[Test build #118999 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118999/testR

[GitHub] [spark] SparkQA commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
SparkQA commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#issuecomment-591776664 **[Test build #119008 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119008/testReport)

[GitHub] [spark] SparkQA commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty

2020-02-26 Thread GitBox
SparkQA commented on issue #27693: [SPARK-30947][CORE] Log better message when accelerate resource is empty URL: https://github.com/apache/spark/pull/27693#issuecomment-591776565 **[Test build #119000 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119000/t

[GitHub] [spark] AmplabJenkins removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591775816 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591775811 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591775816 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berke

[GitHub] [spark] AmplabJenkins commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591775811 Merged build finished. Test FAILed. This is an auto

[GitHub] [spark] SparkQA removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591744303 **[Test build #119004 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119004/testR

[GitHub] [spark] SparkQA commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query

2020-02-26 Thread GitBox
SparkQA commented on issue #27567: [SPARK-30822][SQL] Remove semicolon at the end of a sql query URL: https://github.com/apache/spark/pull/27567#issuecomment-591775692 **[Test build #119004 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/119004/testReport)*

[GitHub] [spark] AmplabJenkins removed a comment on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#issuecomment-591775139 Merged build finished. Test PASSed. This

[GitHub] [spark] cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384908893 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalys

[GitHub] [spark] AmplabJenkins removed a comment on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#issuecomment-591775146 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384908705 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalys

[GitHub] [spark] cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384908705 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalys

[GitHub] [spark] AmplabJenkins commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#issuecomment-591775146 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.be

[GitHub] [spark] AmplabJenkins commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#issuecomment-591775139 Merged build finished. Test PASSed. This is an au

[GitHub] [spark] cloud-fan commented on a change in pull request #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#discussion_r384908202 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeFormatt

[GitHub] [spark] cloud-fan commented on a change in pull request #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter

2020-02-26 Thread GitBox
cloud-fan commented on a change in pull request #27707: [SPARK-30958][SQL] do not set default era for DateTimeFormatter URL: https://github.com/apache/spark/pull/27707#discussion_r384649943 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/TimestampFormat

[GitHub] [spark] AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591773253 Test PASSed. Refer to this link for build results (access rights to CI server needed)

[GitHub] [spark] AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591773245 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#issuecomment-591773276 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#issuecomment-591773281 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#issuecomment-591773281 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591773253 Test PASSed. Refer to this link for build results (access rights to CI server needed): ht

[GitHub] [spark] AmplabJenkins commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#issuecomment-591773276 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591773245 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
SparkQA commented on issue #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#issuecomment-591773039 **[Test build #119006 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestB

[GitHub] [spark] imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
imback82 commented on a change in pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718#discussion_r384906600 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst

[GitHub] [spark] SparkQA commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
SparkQA commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591773040 **[Test build #119007 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/

[GitHub] [spark] imback82 opened a new pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table

2020-02-26 Thread GitBox
imback82 opened a new pull request #27718: [WIP][SPARK-30885][SQL][FOLLOW-UP] Fix a hack in ResolveSessionCatalog.TempViewOrV1Table URL: https://github.com/apache/spark/pull/27718 ### What changes were proposed in this pull request? The current approach to resolving temp view

[GitHub] [spark] AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591772276 Test FAILed. Refer to this link for build results (access rights to CI server needed)

[GitHub] [spark] AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591772271 Merged build finished. Test FAILed. ---

[GitHub] [spark] HyukjinKwon commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
HyukjinKwon commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591772533 retest this please This is an aut

[GitHub] [spark] AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591772276 Test FAILed. Refer to this link for build results (access rights to CI server needed): htt

[GitHub] [spark] AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591772271 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
SparkQA commented on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591771931 **[Test build #119001 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder

[GitHub] [spark] SparkQA removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27717: [SPARK-23435][INFRA][FOLLOW-UP] Remove unnecessary dependency listing in AppVeyor URL: https://github.com/apache/spark/pull/27717#issuecomment-591733602 **[Test build #119001 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequest

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27702: [SPARK-30955][SQL] Exclude Generate output when aliasing in nested column pruning

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27702: [SPARK-30955][SQL] Exclude Generate output when aliasing in nested column pruning URL: https://github.com/apache/spark/pull/27702#discussion_r384904993 ## File path: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuit

[GitHub] [spark] beliefer commented on issue #27674: [SPARK-30888][CORE][DOC] Add version information to the configuration of Network

2020-02-26 Thread GitBox
beliefer commented on issue #27674: [SPARK-30888][CORE][DOC] Add version information to the configuration of Network URL: https://github.com/apache/spark/pull/27674#issuecomment-591766829 @HyukjinKwon Thanks for your help! Th

[GitHub] [spark] beliefer commented on issue #27708: [SPARK-30910][CORE][DOC] Add version information to the configuration of R

2020-02-26 Thread GitBox
beliefer commented on issue #27708: [SPARK-30910][CORE][DOC] Add version information to the configuration of R URL: https://github.com/apache/spark/pull/27708#issuecomment-591766679 @HyukjinKwon Thanks for your help! This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591765542 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591765538 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591765538 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591765542 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591705625 **[Test build #118996 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11899

[GitHub] [spark] SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591765007 **[Test build #118996 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118996/testR

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591764366 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591764366 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591764356 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591764356 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591764096 **[Test build #118998 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118998/testR

[GitHub] [spark] SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591724726 **[Test build #118998 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11899

[GitHub] [spark] SaurabhChawla100 commented on a change in pull request #27636: [SPARK-30873][CORE][YARN]Handling Node Decommissioning for Yarn cluster manger in Spark

2020-02-26 Thread GitBox
SaurabhChawla100 commented on a change in pull request #27636: [SPARK-30873][CORE][YARN]Handling Node Decommissioning for Yarn cluster manger in Spark URL: https://github.com/apache/spark/pull/27636#discussion_r384895513 ## File path: core/src/main/scala/org/apache/spark/internal/c

[GitHub] [spark] dongjoon-hyun closed pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation

2020-02-26 Thread GitBox
dongjoon-hyun closed pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation URL: https://github.com/apache/spark/pull/27715 This is an automated message from the Apache Git Service. To respon

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591759552 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591759545 Merged build finished. Test PASSed.

[GitHub] [spark] dongjoon-hyun commented on issue #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation

2020-02-26 Thread GitBox
dongjoon-hyun commented on issue #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation URL: https://github.com/apache/spark/pull/27715#issuecomment-591760107 Thank you, @HyukjinKwon and @viirya . Since this is the first attempt to recover this test coverage, we can imp

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation URL: https://github.com/apache/spark/pull/27715#discussion_r384893853 ## File path: .github/workflows/master.yml ## @@ -117,3 +117,40 @@ jobs:

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591759545 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
AmplabJenkins commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591759552 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation

2020-02-26 Thread GitBox
dongjoon-hyun commented on a change in pull request #27715: [SPARK-30963][INFRA] Add GitHub Action job for document generation URL: https://github.com/apache/spark/pull/27715#discussion_r384893853 ## File path: .github/workflows/master.yml ## @@ -117,3 +117,40 @@ jobs:

[GitHub] [spark] SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA commented on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591758835 **[Test build #118994 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118994/testR

[GitHub] [spark] SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index

2020-02-26 Thread GitBox
SparkQA removed a comment on issue #27716: [SPARK-30964][Core][WebUI] Accelerate InMemoryStore with a new index URL: https://github.com/apache/spark/pull/27716#issuecomment-591699413 **[Test build #118994 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11899

<    1   2   3   4   5   6   7   8   9   10   >