[GitHub] [spark] AmplabJenkins removed a comment on pull request #28901: [SPARK-32064][SQL] Supporting create temporary table

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28901: URL: https://github.com/apache/spark/pull/28901#issuecomment-654644370 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654644220 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA removed a comment on pull request #28996: [SPARK-29358][SQL] Make unionByName optionally fill missing columns with nulls

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28996: URL: https://github.com/apache/spark/pull/28996#issuecomment-654510421 **[Test build #125141 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125141/testReport)** for PR 28996 at commit

[GitHub] [spark] SparkQA removed a comment on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654624795 **[Test build #125181 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125181/testReport)** for PR 28998 at commit

[GitHub] [spark] SparkQA removed a comment on pull request #28901: [SPARK-32064][SQL] Supporting create temporary table

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28901: URL: https://github.com/apache/spark/pull/28901#issuecomment-654617568 **[Test build #125178 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125178/testReport)** for PR 28901 at commit

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654644223 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on pull request #28901: [SPARK-32064][SQL] Supporting create temporary table

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28901: URL: https://github.com/apache/spark/pull/28901#issuecomment-654644370 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins commented on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654644134 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654644134 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654644368 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654644368 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] SparkQA removed a comment on pull request #28965: [SPARK-32124][CORE][FOLLOW-UP] Use the invalid value Int.MinValue to fill the map index when the event logs from the old Spark versio

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28965: URL: https://github.com/apache/spark/pull/28965#issuecomment-654584765 **[Test build #125171 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125171/testReport)** for PR 28965 at commit

[GitHub] [spark] SparkQA commented on pull request #28904: [SPARK-30462][SS] Streamline the logic on file stream source and sink to avoid memory issue

2020-07-07 Thread GitBox
SparkQA commented on pull request #28904: URL: https://github.com/apache/spark/pull/28904#issuecomment-654644092 **[Test build #125172 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125172/testReport)** for PR 28904 at commit

[GitHub] [spark] SparkQA commented on pull request #28965: [SPARK-32124][CORE][FOLLOW-UP] Use the invalid value Int.MinValue to fill the map index when the event logs from the old Spark version

2020-07-07 Thread GitBox
SparkQA commented on pull request #28965: URL: https://github.com/apache/spark/pull/28965#issuecomment-654644086 **[Test build #125171 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125171/testReport)** for PR 28965 at commit

[GitHub] [spark] SparkQA commented on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
SparkQA commented on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654644080 **[Test build #125183 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125183/testReport)** for PR 29019 at commit

[GitHub] [spark] SparkQA commented on pull request #26935: [SPARK-30294][SS] Explicitly defines read-only StateStore and optimize for HDFSBackedStateStore

2020-07-07 Thread GitBox
SparkQA commented on pull request #26935: URL: https://github.com/apache/spark/pull/26935#issuecomment-654644076 **[Test build #125176 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125176/testReport)** for PR 26935 at commit

[GitHub] [spark] SparkQA commented on pull request #28859: [SPARK-32024][WEBUI] Update ApplicationStoreInfo.size during HistoryServerDiskManager initializing

2020-07-07 Thread GitBox
SparkQA commented on pull request #28859: URL: https://github.com/apache/spark/pull/28859#issuecomment-654644083 **[Test build #125173 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125173/testReport)** for PR 28859 at commit

[GitHub] [spark] SparkQA commented on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
SparkQA commented on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654644082 **[Test build #125180 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125180/testReport)** for PR 28999 at commit

[GitHub] [spark] SparkQA commented on pull request #27428: [SPARK-30276][SQL] Support Filter expression allows simultaneous use of DISTINCT

2020-07-07 Thread GitBox
SparkQA commented on pull request #27428: URL: https://github.com/apache/spark/pull/27428#issuecomment-654644081 **[Test build #125165 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125165/testReport)** for PR 27428 at commit

[GitHub] [spark] SparkQA commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
SparkQA commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654644098 **[Test build #125179 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125179/testReport)** for PR 29012 at commit

[GitHub] [spark] SparkQA commented on pull request #28901: [SPARK-32064][SQL] Supporting create temporary table

2020-07-07 Thread GitBox
SparkQA commented on pull request #28901: URL: https://github.com/apache/spark/pull/28901#issuecomment-654644075 **[Test build #125178 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125178/testReport)** for PR 28901 at commit

[GitHub] [spark] SparkQA commented on pull request #28988: [SPARK-32163][SQL] Nested pruning should work even with cosmetic variations

2020-07-07 Thread GitBox
SparkQA commented on pull request #28988: URL: https://github.com/apache/spark/pull/28988#issuecomment-654644087 **[Test build #125166 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125166/testReport)** for PR 28988 at commit

[GitHub] [spark] SparkQA commented on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
SparkQA commented on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654644078 **[Test build #125181 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125181/testReport)** for PR 28998 at commit

[GitHub] [spark] SparkQA commented on pull request #28969: [SPARK-32150][BUILD] Upgrade to ZStd 1.4.5-4

2020-07-07 Thread GitBox
SparkQA commented on pull request #28969: URL: https://github.com/apache/spark/pull/28969#issuecomment-654644079 **[Test build #125177 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125177/testReport)** for PR 28969 at commit

[GitHub] [spark] SparkQA commented on pull request #28833: [SPARK-20680][SQL] Spark-sql do not support for creating table with void column datatype

2020-07-07 Thread GitBox
SparkQA commented on pull request #28833: URL: https://github.com/apache/spark/pull/28833#issuecomment-654644077 **[Test build #125164 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125164/testReport)** for PR 28833 at commit

[GitHub] [spark] SparkQA commented on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
SparkQA commented on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654644093 **[Test build #125167 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125167/testReport)** for PR 28979 at commit

[GitHub] [spark] SparkQA commented on pull request #28996: [SPARK-29358][SQL] Make unionByName optionally fill missing columns with nulls

2020-07-07 Thread GitBox
SparkQA commented on pull request #28996: URL: https://github.com/apache/spark/pull/28996#issuecomment-654644097 **[Test build #125141 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125141/testReport)** for PR 28996 at commit

[GitHub] [spark] SparkQA commented on pull request #28808: [SPARK-31975][SQL] Show AnalysisException when WindowFunction is used without WindowExpression

2020-07-07 Thread GitBox
SparkQA commented on pull request #28808: URL: https://github.com/apache/spark/pull/28808#issuecomment-654644091 **[Test build #125175 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125175/testReport)** for PR 28808 at commit

[GitHub] [spark] dilipbiswal commented on a change in pull request #28683: [SPARK-31875][SQL] Provide a option to disable user supplied Hints

2020-07-07 Thread GitBox
dilipbiswal commented on a change in pull request #28683: URL: https://github.com/apache/spark/pull/28683#discussion_r450650126 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveHints.scala ## @@ -278,4 +278,15 @@ object ResolveHints {

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654641889 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654641878 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] dilipbiswal commented on a change in pull request #28683: [SPARK-31875][SQL] Provide a option to disable user supplied Hints

2020-07-07 Thread GitBox
dilipbiswal commented on a change in pull request #28683: URL: https://github.com/apache/spark/pull/28683#discussion_r450650126 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveHints.scala ## @@ -278,4 +278,15 @@ object ResolveHints {

[GitHub] [spark] AmplabJenkins commented on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654641878 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] dilipbiswal commented on a change in pull request #28953: [SPARK-32013][SQL] Support query execution before/after reading/writing DataFrame over JDBC

2020-07-07 Thread GitBox
dilipbiswal commented on a change in pull request #28953: URL: https://github.com/apache/spark/pull/28953#discussion_r450648679 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala ## @@ -122,6 +122,37 @@ object JdbcUtils

[GitHub] [spark] SparkQA removed a comment on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654556154 **[Test build #125161 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125161/testReport)** for PR 28979 at commit

[GitHub] [spark] SparkQA commented on pull request #28979: [SPARK-32154][SQL] Use ExpressionEncoder for the return type of ScalaUDF to convert to catalyst type

2020-07-07 Thread GitBox
SparkQA commented on pull request #28979: URL: https://github.com/apache/spark/pull/28979#issuecomment-654640655 **[Test build #125161 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125161/testReport)** for PR 28979 at commit

[GitHub] [spark] AmplabJenkins removed a comment on pull request #26935: [SPARK-30294][SS] Explicitly defines read-only StateStore and optimize for HDFSBackedStateStore

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

[GitHub] [spark] HyukjinKwon commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
HyukjinKwon commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654638263 I think it's just that the Jenkins machine became extremely slow for an unknown reason. This is an

[GitHub] [spark] AmplabJenkins commented on pull request #26935: [SPARK-30294][SS] Explicitly defines read-only StateStore and optimize for HDFSBackedStateStore

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #26935: URL: https://github.com/apache/spark/pull/26935#issuecomment-654638195 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654635334 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 #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
SparkQA commented on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654637734 **[Test build #125183 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125183/testReport)** for PR 29019 at commit

[GitHub] [spark] cloud-fan commented on a change in pull request #28683: [SPARK-31875][SQL] Provide a option to disable user supplied Hints

2020-07-07 Thread GitBox
cloud-fan commented on a change in pull request #28683: URL: https://github.com/apache/spark/pull/28683#discussion_r450643675 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveHints.scala ## @@ -278,4 +278,15 @@ object ResolveHints {

[GitHub] [spark] cloud-fan commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
cloud-fan commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654636022 When did we disable SPARK-27460 ? I'm not aware of it. This is an automated message from the Apache Git

[GitHub] [spark] AmplabJenkins commented on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654635334 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] Ngone51 edited a comment on pull request #28850: [SPARK-32015][Core]Remote inheritable thread local variables after spark context is stopped

2020-07-07 Thread GitBox
Ngone51 edited a comment on pull request #28850: URL: https://github.com/apache/spark/pull/28850#issuecomment-654633882 I agree with @srowen and think the latest update is still too hacky. I'm thinking about another way that we might avoid passing `activeThreadSession` to threads

[GitHub] [spark] cloud-fan commented on pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
cloud-fan commented on pull request #29019: URL: https://github.com/apache/spark/pull/29019#issuecomment-654634847 cc @dongjoon-hyun @viirya This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] Ngone51 edited a comment on pull request #28850: [SPARK-32015][Core]Remote inheritable thread local variables after spark context is stopped

2020-07-07 Thread GitBox
Ngone51 edited a comment on pull request #28850: URL: https://github.com/apache/spark/pull/28850#issuecomment-654633882 I agree with @srowen and think the latest update is still too hacky. I'm thinking about another way that we might avoid passing `activeThreadSession` to threads

[GitHub] [spark] cloud-fan opened a new pull request #29019: [2.4][SPARK-32167][SQL] Fix GetArrayStructFields to respect inner field's nullability together

2020-07-07 Thread GitBox
cloud-fan opened a new pull request #29019: URL: https://github.com/apache/spark/pull/29019 Backport https://github.com/apache/spark/pull/28992 to 2.4 This is an automated message from the Apache Git Service. To respond to

[GitHub] [spark] Ngone51 commented on pull request #28850: [SPARK-32015][Core]Remote inheritable thread local variables after spark context is stopped

2020-07-07 Thread GitBox
Ngone51 commented on pull request #28850: URL: https://github.com/apache/spark/pull/28850#issuecomment-654633882 I agree with @srowen and think the latest update is still too hacky. I'm thinking about another way that we might avoid passing `activeThreadSession` to threads that are

[GitHub] [spark] WinkerDu commented on pull request #29000: [SPARK-27194][SPARK-29302][SQL] Fix commit collision in dynamic partition overwrite mode

2020-07-07 Thread GitBox
WinkerDu commented on pull request #29000: URL: https://github.com/apache/spark/pull/29000#issuecomment-654632940 @turboFei Thanks for your comments. Actually I think there is no partition-explosion cost. Commit task output dir is generated by `FileFormatDataWriter.newOutputWriter`, no

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28924: [SPARK-32091][CORE] Ignore timeout error when remove blocks on the lost executor

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28924: URL: https://github.com/apache/spark/pull/28924#issuecomment-654631941 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28924: [SPARK-32091][CORE] Ignore timeout error when remove blocks on the lost executor

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28924: URL: https://github.com/apache/spark/pull/28924#issuecomment-654631932 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on pull request #28924: [SPARK-32091][CORE] Ignore timeout error when remove blocks on the lost executor

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28924: URL: https://github.com/apache/spark/pull/28924#issuecomment-654631932 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] SparkQA commented on pull request #28924: [SPARK-32091][CORE] Ignore timeout error when remove blocks on the lost executor

2020-07-07 Thread GitBox
SparkQA commented on pull request #28924: URL: https://github.com/apache/spark/pull/28924#issuecomment-654631366 **[Test build #125169 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125169/testReport)** for PR 28924 at commit

[GitHub] [spark] SparkQA removed a comment on pull request #28924: [SPARK-32091][CORE] Ignore timeout error when remove blocks on the lost executor

2020-07-07 Thread GitBox
SparkQA removed a comment on pull request #28924: URL: https://github.com/apache/spark/pull/28924#issuecomment-654573918 **[Test build #125169 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125169/testReport)** for PR 28924 at commit

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28269: [SPARK-31493][SQL] Optimize InSet to In according partition size at InSubqueryExec

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28269: URL: https://github.com/apache/spark/pull/28269#issuecomment-654630197 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 #28269: [SPARK-31493][SQL] Optimize InSet to In according partition size at InSubqueryExec

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28269: URL: https://github.com/apache/spark/pull/28269#issuecomment-654630197 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29006: URL: https://github.com/apache/spark/pull/29006#issuecomment-654627110 Merged build finished. Test FAILed. This is an automated message from the Apache Git Service. To

[GitHub] [spark] igreenfield commented on pull request #28629: [SPARK-31769][CORE] Add MDC support for driver threads

2020-07-07 Thread GitBox
igreenfield commented on pull request #28629: URL: https://github.com/apache/spark/pull/28629#issuecomment-654627300 @cloud-fan, What should we do if `setLocalProperties` and inherited MDC has same keys? which will take effect?

[GitHub] [spark] AmplabJenkins commented on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #29006: URL: https://github.com/apache/spark/pull/29006#issuecomment-654627110 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] yaooqinn closed pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
yaooqinn closed pull request #29006: URL: https://github.com/apache/spark/pull/29006 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [spark] yaooqinn commented on a change in pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
yaooqinn commented on a change in pull request #29006: URL: https://github.com/apache/spark/pull/29006#discussion_r450633892 ## File path: pom.xml ## @@ -3085,6 +3085,31 @@ nohive 3.2.10 + + + +

[GitHub] [spark] HyukjinKwon commented on a change in pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
HyukjinKwon commented on a change in pull request #29006: URL: https://github.com/apache/spark/pull/29006#discussion_r450633583 ## File path: pom.xml ## @@ -3085,6 +3085,31 @@ nohive 3.2.10 + + + +

[GitHub] [spark] yaooqinn commented on a change in pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
yaooqinn commented on a change in pull request #29006: URL: https://github.com/apache/spark/pull/29006#discussion_r450633128 ## File path: pom.xml ## @@ -3085,6 +3085,31 @@ nohive 3.2.10 + + + +

[GitHub] [spark] yaooqinn commented on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29006: [SPARK-32058][BUILD][SQL][test-hive1.2][test-hadoop2.7][FOLLOWUP] Set hadoop 2.7.4 for hive 1.2 profile

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29006: URL: https://github.com/apache/spark/pull/29006#issuecomment-654322005 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] turboFei commented on pull request #28989: [SPARK-27194][SPARK-29302][SQL] Define a spark staging committer to resolve FileAlreadyExistingException

2020-07-07 Thread GitBox
turboFei commented on pull request #28989: URL: https://github.com/apache/spark/pull/28989#issuecomment-654625920 will try to fix it. This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654625124 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 #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654625180 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654625102 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 #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654625180 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 #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654625124 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654610268 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
AmplabJenkins commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654625102 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
AmplabJenkins removed a comment on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654608665 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
SparkQA commented on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654624795 **[Test build #125181 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125181/testReport)** for PR 28998 at commit

[GitHub] [spark] SparkQA commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
SparkQA commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654624788 **[Test build #125179 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125179/testReport)** for PR 29012 at commit

[GitHub] [spark] SparkQA commented on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
SparkQA commented on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654624692 **[Test build #125180 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125180/testReport)** for PR 28999 at commit

[GitHub] [spark] MaxGekk commented on pull request #28999: [MINOR][SQL] Re-use GetTimestamp in ParseToDate

2020-07-07 Thread GitBox
MaxGekk commented on pull request #28999: URL: https://github.com/apache/spark/pull/28999#issuecomment-654623188 jenkins, retest this, please This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] dongjoon-hyun commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
dongjoon-hyun commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654622914 Hi, @gengliangwang and @gatorsmile . SBT PRBuilder seems to be running serially and took `8 hr 26 min`. Do you think something is changed inside Jenkins? It seems

[GitHub] [spark] MaxGekk commented on pull request #28998: [SPARK-32173][SQL] Deduplicate code in FromUTCTimestamp and ToUTCTimestamp

2020-07-07 Thread GitBox
MaxGekk commented on pull request #28998: URL: https://github.com/apache/spark/pull/28998#issuecomment-654622974 jenkins, retest this, please This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] MaxGekk commented on pull request #29012: [SPARK-31710][SQL][FOLLOWUP] Allow cast numeric to timestamp by default

2020-07-07 Thread GitBox
MaxGekk commented on pull request #29012: URL: https://github.com/apache/spark/pull/29012#issuecomment-654622656 jenkins, retest this, please This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #28683: [SPARK-31875][SQL] Provide a option to disable user supplied Hints

2020-07-07 Thread GitBox
dongjoon-hyun commented on a change in pull request #28683: URL: https://github.com/apache/spark/pull/28683#discussion_r450628644 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ## @@ -2106,6 +2106,15 @@ object SQLConf {

[GitHub] [spark] viirya commented on a change in pull request #28996: [SPARK-29358][SQL] Make unionByName optionally fill missing columns with nulls

2020-07-07 Thread GitBox
viirya commented on a change in pull request #28996: URL: https://github.com/apache/spark/pull/28996#discussion_r450312370 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ## @@ -2656,6 +2656,14 @@ object SQLConf { .checkValue(_ >

<    9   10   11   12   13   14