[GitHub] [spark] SparkQA commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed

2020-02-20 Thread GitBox
SparkQA commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed URL: https://github.com/apache/spark/pull/27644#issuecomment-589541503 **[Test build #118756 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118756/testReport)**

[GitHub] [spark] SparkQA removed a comment on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed

2020-02-20 Thread GitBox
SparkQA removed a comment on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed URL: https://github.com/apache/spark/pull/27644#issuecomment-589510776 **[Test build #118756 has

[GitHub] [spark] AmplabJenkins commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed URL: https://github.com/apache/spark/pull/27644#issuecomment-589541605 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed URL: https://github.com/apache/spark/pull/27644#issuecomment-589541605 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27644: [SPARK-30868][SQL] Throw Exception if runHive(sql) failed URL: https://github.com/apache/spark/pull/27644#issuecomment-589541612 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on issue #27495: [SPARK-28880][SQL] Support ANSI nested bracketed comments

2020-02-20 Thread GitBox
cloud-fan commented on issue #27495: [SPARK-28880][SQL] Support ANSI nested bracketed comments URL: https://github.com/apache/spark/pull/27495#issuecomment-589540986 I have a new idea. Without considering hint, the rule to match comment is very simple: ``` '/*'

[GitHub] [spark] AmplabJenkins commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589539592 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589539598 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589539598 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589539592 Merged build finished. Test PASSed. This is an

[GitHub] [spark] SparkQA removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
SparkQA removed a comment on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589493045 **[Test build #118749 has

[GitHub] [spark] SparkQA commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs

2020-02-20 Thread GitBox
SparkQA commented on issue #27560: [SPARK-30809][SQL] Review and fix issues in SQL API docs URL: https://github.com/apache/spark/pull/27560#issuecomment-589538924 **[Test build #118749 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118749/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#issuecomment-589538601 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#issuecomment-589538601 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#issuecomment-589538595 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#issuecomment-589538595 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
SparkQA commented on issue #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#issuecomment-589538273 **[Test build #118763 has

[GitHub] [spark] HyukjinKwon closed pull request #27661: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
HyukjinKwon closed pull request #27661: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27661 This is an automated message from the Apache Git Service.

[GitHub] [spark] peter-toth commented on a change in pull request #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get

2020-02-20 Thread GitBox
peter-toth commented on a change in pull request #27655: [SPARK-30897][SQL] The behavior of ArrayExists should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27655#discussion_r382435704 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#discussion_r382433518 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#issuecomment-589534209 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#issuecomment-589534205 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#issuecomment-589534205 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#issuecomment-589534209 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] cloud-fan closed pull request #27658: [SPARK-30894][SQL] Make Size's nullable independent from SQL config changes

2020-02-20 Thread GitBox
cloud-fan closed pull request #27658: [SPARK-30894][SQL] Make Size's nullable independent from SQL config changes URL: https://github.com/apache/spark/pull/27658 This is an automated message from the Apache Git Service. To

[GitHub] [spark] cloud-fan commented on issue #27658: [SPARK-30894][SQL] Make Size's nullable independent from SQL config changes

2020-02-20 Thread GitBox
cloud-fan commented on issue #27658: [SPARK-30894][SQL] Make Size's nullable independent from SQL config changes URL: https://github.com/apache/spark/pull/27658#issuecomment-589534103 thanks, merging to master/3.0! This is

[GitHub] [spark] SparkQA commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
SparkQA commented on issue #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#issuecomment-589533846 **[Test build #118762 has

[GitHub] [spark] Ngone51 commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
Ngone51 commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#discussion_r382432246 ## File path:

[GitHub] [spark] cloud-fan commented on issue #27627: [WIP][SPARK-28067][SQL] Fix incorrect results for decimal aggregate sum by returning null on decimal overflow

2020-02-20 Thread GitBox
cloud-fan commented on issue #27627: [WIP][SPARK-28067][SQL] Fix incorrect results for decimal aggregate sum by returning null on decimal overflow URL: https://github.com/apache/spark/pull/27627#issuecomment-589530921 > Sum does not take care of possibility of overflow for the intermediate

[GitHub] [spark] AmplabJenkins commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589529896 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589529900 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589529900 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589529896 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] SparkQA removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
SparkQA removed a comment on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589468236 **[Test build #118743 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118743/testReport)** for PR

[GitHub] [spark] SparkQA commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
SparkQA commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589529295 **[Test build #118743 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118743/testReport)** for PR 27662 at

[GitHub] [spark] AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589526227 Test FAILed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589526227 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589526219 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589526219 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
SparkQA removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589521877 **[Test build #118761 has

[GitHub] [spark] SparkQA commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
SparkQA commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589526122 **[Test build #118761 has

[GitHub] [spark] Ngone51 commented on issue #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default

2020-02-20 Thread GitBox
Ngone51 commented on issue #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default URL: https://github.com/apache/spark/pull/27488#issuecomment-589525950 thanks all! This is an automated

[GitHub] [spark] Ngone51 commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
Ngone51 commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589526023 thanks all! This is an automated message from the Apache Git Service. To

[GitHub] [spark] Ngone51 commented on issue #27540: Revert "[SPARK-28093][SQL] Fix TRIM/LTRIM/RTRIM function parameter order issue"

2020-02-20 Thread GitBox
Ngone51 commented on issue #27540: Revert "[SPARK-28093][SQL] Fix TRIM/LTRIM/RTRIM function parameter order issue" URL: https://github.com/apache/spark/pull/27540#issuecomment-589525865 thanks all!! This is an automated

[GitHub] [spark] cloud-fan commented on a change in pull request #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#discussion_r382423049 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#discussion_r382422992 ## File path:

[GitHub] [spark] HyukjinKwon commented on issue #27661: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
HyukjinKwon commented on issue #27661: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27661#issuecomment-589523563 Let's close and review #27618 This

[GitHub] [spark] cloud-fan commented on issue #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default

2020-02-20 Thread GitBox
cloud-fan commented on issue #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default URL: https://github.com/apache/spark/pull/27488#issuecomment-589522387 thanks, merging to master/3.0!

[GitHub] [spark] cloud-fan closed pull request #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default

2020-02-20 Thread GitBox
cloud-fan closed pull request #27488: [SPARK-26580][SQL][ML][FOLLOW-UP] Throw exception when use untyped UDF by default URL: https://github.com/apache/spark/pull/27488 This is an automated message from the Apache Git

[GitHub] [spark] SparkQA commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
SparkQA commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589521877 **[Test build #118761 has

[GitHub] [spark] Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block

2020-02-20 Thread GitBox
Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block URL: https://github.com/apache/spark/pull/27604#discussion_r382411144 ## File path:

[GitHub] [spark] Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block

2020-02-20 Thread GitBox
Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block URL: https://github.com/apache/spark/pull/27604#discussion_r382412090 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589520239 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589520236 Merged build finished. Test PASSed.

[GitHub] [spark] Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block

2020-02-20 Thread GitBox
Ngone51 commented on a change in pull request #27604: [SPARK-30849][CORE][SHUFFLE]Fix application failed due to failed to get MapStatuses broadcast block URL: https://github.com/apache/spark/pull/27604#discussion_r382410631 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589520236 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27657: [SPARK-30899]: CreateArray/CreateMap's data type should not depend on SQLConf.get URL: https://github.com/apache/spark/pull/27657#issuecomment-589520239 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
SparkQA commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589519875 **[Test build #118760 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589518224 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589518226 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589518224 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589518226 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
cloud-fan commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589517992 retest this please This is an automated

[GitHub] [spark] cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#discussion_r382416037 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589516919 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589516913 Merged build finished. Test FAILed.

[GitHub] [spark] cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#discussion_r382415458 ## File path:

[GitHub] [spark] SparkQA commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
SparkQA commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589516807 **[Test build #118753 has

[GitHub] [spark] AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589516919 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#discussion_r382415344 ## File path:

[GitHub] [spark] SparkQA removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
SparkQA removed a comment on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589499680 **[Test build #118753 has

[GitHub] [spark] AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27659: [SPARK-30906][SQL] Turning off AQE in CacheManager is not thread-safe URL: https://github.com/apache/spark/pull/27659#issuecomment-589516913 Merged build finished. Test FAILed. This

[GitHub] [spark] maropu commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
maropu commented on issue #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662#issuecomment-589516586 Thanks! Merged to master/3.0. This is an automated message from the Apache Git

[GitHub] [spark] AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589516320 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589516325 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#issuecomment-589516297 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#issuecomment-589516294 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589516325 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#issuecomment-589516294 Merged build finished. Test PASSed.

[GitHub] [spark] maropu closed pull request #27662: [MINOR][SQL] Fix error position of NOSCAN

2020-02-20 Thread GitBox
maropu closed pull request #27662: [MINOR][SQL] Fix error position of NOSCAN URL: https://github.com/apache/spark/pull/27662 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 issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#issuecomment-589516297 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589516320 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#discussion_r382414618 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/TimestampFormatter.scala ##

[GitHub] [spark] SparkQA commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision

2020-02-20 Thread GitBox
SparkQA commented on issue #27618: [SPARK-30869][SQL] Convert dates to/from timestamps in microseconds precision URL: https://github.com/apache/spark/pull/27618#issuecomment-589516028 **[Test build #118758 has

[GitHub] [spark] SparkQA commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
SparkQA commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589516013 **[Test build #118759 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118759/testReport)** for PR 27617 at

[GitHub] [spark] cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#discussion_r382414377 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala ## @@

[GitHub] [spark] maropu commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns

2020-02-20 Thread GitBox
maropu commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns URL: https://github.com/apache/spark/pull/27651#discussion_r382414058 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns

2020-02-20 Thread GitBox
maropu commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns URL: https://github.com/apache/spark/pull/27651#discussion_r382414020 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#discussion_r382413997 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala ## @@

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

2020-02-20 Thread GitBox
iRakson commented on issue #27580: [SPARK-27619][SQL]MapType should be prohibited in hash expressions URL: https://github.com/apache/spark/pull/27580#issuecomment-589515559 retest this please This is an automated message

[GitHub] [spark] cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#discussion_r382413974 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala ## @@

[GitHub] [spark] iRakson commented on issue #27648: [SPARK-24994][SQL] : Support filter pushdown for short and byte without explicit casting

2020-02-20 Thread GitBox
iRakson commented on issue #27648: [SPARK-24994][SQL] : Support filter pushdown for short and byte without explicit casting URL: https://github.com/apache/spark/pull/27648#issuecomment-589515318 > Is that pushdown legal? if `id` is short, `id = 2` is parsed as `cast(id as int) = 2`.

[GitHub] [spark] AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589512791 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589512797 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins removed a comment on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589512791 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] cloud-fan commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27651: [SPARK-30903][SQL] Fail fast on duplicate columns when analyze columns URL: https://github.com/apache/spark/pull/27651#discussion_r382411322 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
AmplabJenkins commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589512797 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

2020-02-20 Thread GitBox
SparkQA commented on issue #27617: [SPARK-30865][SQL] Refactor DateTimeUtils URL: https://github.com/apache/spark/pull/27617#issuecomment-589512485 **[Test build #118757 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118757/testReport)** for PR 27617 at

[GitHub] [spark] cloud-fan commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table

2020-02-20 Thread GitBox
cloud-fan commented on a change in pull request #27597: [SPARK-30844][SQL]Static partition should also follow StoreAssignmentPolicy when insert into table URL: https://github.com/apache/spark/pull/27597#discussion_r382410965 ## File path:

[GitHub] [spark] maropu commented on issue #27648: [SPARK-24994][SQL] : Support filter pushdown for short and byte without explicit casting

2020-02-20 Thread GitBox
maropu commented on issue #27648: [SPARK-24994][SQL] : Support filter pushdown for short and byte without explicit casting URL: https://github.com/apache/spark/pull/27648#issuecomment-589512300 > 2 is within short range still filter is not pushed down. > But it pushes filter if we cast

  1   2   3   4   5   6   7   8   9   10   >