[GitHub] [spark] AmplabJenkins commented on pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28123: URL: https://github.com/apache/spark/pull/28123#issuecomment-643109360 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 #27617: [SPARK-30865][SQL] Refactor DateTimeUtils

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

[GitHub] [spark] sarutak commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak commented on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643116427 > The infinite drawing issue seems also fixed if the zoom is disabled as default.

[GitHub] [spark] sarutak edited a comment on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak edited a comment on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643116427 > The infinite drawing issue seems also fixed if the zoom is disabled as default. Actually that is not fixed in 4.21.0. I replaced `vis` included in this PR

[GitHub] [spark] sarutak edited a comment on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak edited a comment on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643116427 > The infinite drawing issue seems also fixed if the zoom is disabled as default. Actually this is not fixed in 4.21.0. I replaced `vis` included in this PR

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

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

[GitHub] [spark] SparkQA removed a comment on pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

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

[GitHub] [spark] HyukjinKwon commented on a change in pull request #28556: [SPARK-31736][SQL] Nested column aliasing for RepartitionByExpression/Join

2020-06-12 Thread GitBox
HyukjinKwon commented on a change in pull request #28556: URL: https://github.com/apache/spark/pull/28556#discussion_r439257774 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NestedColumnAliasing.scala ## @@ -34,7 +34,8 @@ object

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

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

[GitHub] [spark] AmplabJenkins commented on pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28808: URL: https://github.com/apache/spark/pull/28808#issuecomment-643121480 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

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

[GitHub] [spark] SparkQA commented on pull request #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

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

[GitHub] [spark] HyukjinKwon commented on a change in pull request #28556: [SPARK-31736][SQL] Nested column aliasing for RepartitionByExpression/Join

2020-06-12 Thread GitBox
HyukjinKwon commented on a change in pull request #28556: URL: https://github.com/apache/spark/pull/28556#discussion_r439257774 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NestedColumnAliasing.scala ## @@ -34,7 +34,8 @@ object

[GitHub] [spark] SparkQA commented on pull request #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] SparkQA commented on pull request #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28801: URL: https://github.com/apache/spark/pull/28801#issuecomment-643184272 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 #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

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

[GitHub] [spark] cloud-fan commented on pull request #28700: [SPARK-31860][BUILD] only push release tags on succes

2020-06-12 Thread GitBox
cloud-fan commented on pull request #28700: URL: https://github.com/apache/spark/pull/28700#issuecomment-643184464 I've reverted it from master/3.0, thanks for the understanding! This is an automated message from the Apache

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28810: [SPARK-31705][SQL][FOLLOWUP] Avoid the unnecessary CNF computation for full-outer joins

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28810: URL: https://github.com/apache/spark/pull/28810#issuecomment-643187453 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 #28810: [SPARK-31705][SQL][FOLLOWUP] Avoid the unnecessary CNF computation for full-outer joins

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28810: URL: https://github.com/apache/spark/pull/28810#issuecomment-643187453 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 #28815: [SPARK-31979] release script should not fail when remove non-existing files

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

[GitHub] [spark] SparkQA commented on pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

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

[GitHub] [spark] wangyum commented on a change in pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

2020-06-12 Thread GitBox
wangyum commented on a change in pull request #28808: URL: https://github.com/apache/spark/pull/28808#discussion_r439337319 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala ## @@ -158,6 +158,9 @@ trait CheckAnalysis extends

[GitHub] [spark] sarutak commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak commented on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643082478 I can't download the downgraded version of `vis` from https://cdnjs.com/libraries/vis. Is it temporarily not available?

[GitHub] [spark] gengliangwang commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
gengliangwang commented on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643083080 @sarutak it works fine in my side: ![image](https://user-images.githubusercontent.com/1097932/84470196-a8006b80-ac37-11ea-81be-a0f28f6c8dc3.png)

[GitHub] [spark] SparkQA commented on pull request #28797: [SPARK-31926][SQL][TEST-HIVE1.2][test-maven] Fix concurrency issue for ThriftCLIService to getPortNumber

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

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

[GitHub] [spark] SparkQA commented on pull request #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] SparkQA commented on pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

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

[GitHub] [spark] SparkQA commented on pull request #28807: [WIP][SPARK-26905][SQL] Follow the SQL:2016 reserved keywords

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

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

[GitHub] [spark] HyukjinKwon commented on pull request #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28641: URL: https://github.com/apache/spark/pull/28641#issuecomment-643123638 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 #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28641: URL: https://github.com/apache/spark/pull/28641#issuecomment-643123638 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

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

[GitHub] [spark] GuoPhilipse commented on pull request #28593: [SPARK-31710][SQL] Fail casting numeric to timestamp by default

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

[GitHub] [spark] zhengruifeng commented on pull request #28710: [SPARK-31893][ML] Add a generic ClassificationSummary trait

2020-06-12 Thread GitBox
zhengruifeng commented on pull request #28710: URL: https://github.com/apache/spark/pull/28710#issuecomment-643131145 @huaxingao since `scoreCol` is only used to compute `BinaryClassificationMetrics`, so what about putting it in `BinaryClassificationSummary`: ``` trait

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28781: [SPARK-31953][SS] Add Spark Structured Streaming History Server Support

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

[GitHub] [spark] TJX2014 opened a new pull request #28814: [SPARK-31968][CORE]Duplicate partition column check when write data

2020-06-12 Thread GitBox
TJX2014 opened a new pull request #28814: URL: https://github.com/apache/spark/pull/28814 ### What changes were proposed in this pull request? A unit test is added Partition duplicate check add in `org.apache.spark.sql.execution.datasources.PartitioningUtils#validatePartitionColumn`

[GitHub] [spark] cloud-fan commented on pull request #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

2020-06-12 Thread GitBox
cloud-fan commented on pull request #28801: URL: https://github.com/apache/spark/pull/28801#issuecomment-643182241 retest this please This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [spark] yqwang-ms commented on pull request #28323: [SPARK-31551][CORE] Fix createSparkUser lost user's non-Hadoop credentials

2020-06-12 Thread GitBox
yqwang-ms commented on pull request #28323: URL: https://github.com/apache/spark/pull/28323#issuecomment-643190127 @tgravescs For safety, I also kept the orignal cloned tokens, see https://github.com/apache/spark/pull/28323/commits/87433b4e33cd5a93785630723f67b7d4f5a0979d PTAL

[GitHub] [spark] AmplabJenkins commented on pull request #27518: [SPARK-30768][SQL] Constraints inferred from inequality attributes

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #27518: URL: https://github.com/apache/spark/pull/27518#issuecomment-643190492 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 #28815: [SPARK-31979] release script should not fail when remove non-existing files

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

[GitHub] [spark] cloud-fan commented on pull request #28787: [SPARK-31959][SQL][test-java11] Fix Gregorian-Julian micros rebasing while switching standard time zone offset

2020-06-12 Thread GitBox
cloud-fan commented on pull request #28787: URL: https://github.com/apache/spark/pull/28787#issuecomment-643086118 More context: To do datetime rebase more efficiently, we put the diffs between two calendars in JSON files, to avoid calculating the diffs every time. We run the

[GitHub] [spark] gengliangwang commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
gengliangwang commented on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643086125 OK, the 4.21.0 package in https://almende.github.io/vis/ should be the same This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

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

[GitHub] [spark] uncleGen commented on pull request #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

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

[GitHub] [spark] wangyum commented on a change in pull request #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

2020-06-12 Thread GitBox
wangyum commented on a change in pull request #28805: URL: https://github.com/apache/spark/pull/28805#discussion_r439236301 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala ## @@ -253,6 +253,58 @@ trait PredicateHelper

[GitHub] [spark] SparkQA removed a comment on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28797: [SPARK-31926][SQL][TEST-HIVE1.2][test-maven] Fix concurrency issue for ThriftCLIService to getPortNumber

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28797: URL: https://github.com/apache/spark/pull/28797#issuecomment-643106210 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 #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

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

[GitHub] [spark] SparkQA removed a comment on pull request #28807: [WIP][SPARK-26905][SQL] Follow the SQL:2016 reserved keywords

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28391: URL: https://github.com/apache/spark/pull/28391#issuecomment-643106049 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 #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28801: URL: https://github.com/apache/spark/pull/28801#issuecomment-643106551 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

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

[GitHub] [spark] AmplabJenkins commented on pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28123: URL: https://github.com/apache/spark/pull/28123#issuecomment-643106630 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 #28797: [SPARK-31926][SQL][TEST-HIVE1.2][test-maven] Fix concurrency issue for ThriftCLIService to getPortNumber

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28802: URL: https://github.com/apache/spark/pull/28802#issuecomment-643106129 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 #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] AmplabJenkins commented on pull request #28593: [SPARK-31710][SQL] Fail casting numeric to timestamp by default

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28593: URL: https://github.com/apache/spark/pull/28593#issuecomment-643106490 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28805: URL: https://github.com/apache/spark/pull/28805#issuecomment-643106070 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 #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] SparkQA commented on pull request #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

2020-06-12 Thread GitBox
SparkQA commented on pull request #28391: URL: https://github.com/apache/spark/pull/28391#issuecomment-643105951 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] AmplabJenkins commented on pull request #28807: [WIP][SPARK-26905][SQL] Follow the SQL:2016 reserved keywords

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28807: URL: https://github.com/apache/spark/pull/28807#issuecomment-643105992 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 #28797: [SPARK-31926][SQL][TEST-HIVE1.2][test-maven] Fix concurrency issue for ThriftCLIService to getPortNumber

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28797: URL: https://github.com/apache/spark/pull/28797#issuecomment-643106210 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 #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28391: URL: https://github.com/apache/spark/pull/28391#issuecomment-643106049 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

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

[GitHub] [spark] SparkQA commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

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

[GitHub] [spark] SparkQA commented on pull request #28593: [SPARK-31710][SQL] Fail casting numeric to timestamp by default

2020-06-12 Thread GitBox
SparkQA commented on pull request #28593: URL: https://github.com/apache/spark/pull/28593#issuecomment-643105946 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] SparkQA commented on pull request #28797: [SPARK-31926][SQL][TEST-HIVE1.2][test-maven] Fix concurrency issue for ThriftCLIService to getPortNumber

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

[GitHub] [spark] SparkQA commented on pull request #28788: [SPARK-31960][Yarn][Build] Only populate Hadoop classpath for no-hadoop build

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

[GitHub] [spark] SparkQA commented on pull request #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

2020-06-12 Thread GitBox
SparkQA commented on pull request #28802: URL: https://github.com/apache/spark/pull/28802#issuecomment-643105941 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [spark] AmplabJenkins commented on pull request #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28805: URL: https://github.com/apache/spark/pull/28805#issuecomment-643106070 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 #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

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

[GitHub] [spark] SparkQA commented on pull request #28801: [SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

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

[GitHub] [spark] AmplabJenkins commented on pull request #28391: [SPARK-31593][SS] Remove unnecessary streaming query progress update

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28391: URL: https://github.com/apache/spark/pull/28391#issuecomment-643106211 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 #28802: [SPARK-31950][SQL][TESTS] Extract SQL keywords from the SqlBase.g4 file

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28802: URL: https://github.com/apache/spark/pull/28802#issuecomment-643106129 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 #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

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

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28805: URL: https://github.com/apache/spark/pull/28805#issuecomment-643123626 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 #28809: [SPARK-31959][SQL][3.0] Fix Gregorian-Julian micros rebasing while switching standard time zone offset

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

[GitHub] [spark] SparkQA commented on pull request #28781: [SPARK-31953][SS] Add Spark Structured Streaming History Server Support

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

[GitHub] [spark] sarutak edited a comment on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak edited a comment on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643129288 LGTM. The updated version seems to work fine. Pending for Jenkins. This is an automated message

[GitHub] [spark] SparkQA commented on pull request #28813: [2.4][SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

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

[GitHub] [spark] SparkQA commented on pull request #28812: [SPARK-31977][SQL] Returns the plan directly from NestedColumnAliasing

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

[GitHub] [spark] Ngone51 commented on a change in pull request #28641: [SPARK-31824][CORE][TESTS] DAGSchedulerSuite: Improve and reuse completeShuffleMapStageSuccessfully

2020-06-12 Thread GitBox
Ngone51 commented on a change in pull request #28641: URL: https://github.com/apache/spark/pull/28641#discussion_r439287499 ## File path: core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ## @@ -889,16 +883,29 @@ class DAGSchedulerSuite extends

[GitHub] [spark] AmplabJenkins commented on pull request #28593: [SPARK-31710][SQL] Fail casting numeric to timestamp by default

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28593: URL: https://github.com/apache/spark/pull/28593#issuecomment-643150529 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 #28593: [SPARK-31710][SQL] Fail casting numeric to timestamp by default

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28593: URL: https://github.com/apache/spark/pull/28593#issuecomment-643150529 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 #28805: [SPARK-28169][SQL] Convert scan predicate condition to CNF

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

[GitHub] [spark] sarutak commented on pull request #28748: [SPARK-30119][WEBUI]Support pagination for streaming tab

2020-06-12 Thread GitBox
sarutak commented on pull request #28748: URL: https://github.com/apache/spark/pull/28748#issuecomment-643172891 Seems OK to me. This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [spark] maropu commented on pull request #28810: [SPARK-31705][SQL][FOLLOWUP] Avoid the unnecessary CNF computation for full-outer joins

2020-06-12 Thread GitBox
maropu commented on pull request #28810: URL: https://github.com/apache/spark/pull/28810#issuecomment-643181366 Yea, sure. 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 #28810: [SPARK-31705][SQL][FOLLOWUP] Avoid the unnecessary CNF computation for full-outer joins

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

[GitHub] [spark] wangyum commented on a change in pull request #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

2020-06-12 Thread GitBox
wangyum commented on a change in pull request #28808: URL: https://github.com/apache/spark/pull/28808#discussion_r439337319 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala ## @@ -158,6 +158,9 @@ trait CheckAnalysis extends

[GitHub] [spark] sarutak commented on pull request #28806: [SPARK-31967][UI] UI Perf regression: Loading jobs UI page takes 40 seconds

2020-06-12 Thread GitBox
sarutak commented on pull request #28806: URL: https://github.com/apache/spark/pull/28806#issuecomment-643093469 @gengliangwang I checked out the change and build but timeline-view wouldn't work.

[GitHub] [spark] AmplabJenkins removed a comment on pull request #28807: [WIP][SPARK-26905][SQL] Follow the SQL:2016 reserved keywords

2020-06-12 Thread GitBox
AmplabJenkins removed a comment on pull request #28807: URL: https://github.com/apache/spark/pull/28807#issuecomment-643099104 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 #28807: [WIP][SPARK-26905][SQL][TESTS] Follow the SQL:2016 reserved keywords

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

[GitHub] [spark] AmplabJenkins commented on pull request #28807: [WIP][SPARK-26905][SQL] Follow the SQL:2016 reserved keywords

2020-06-12 Thread GitBox
AmplabJenkins commented on pull request #28807: URL: https://github.com/apache/spark/pull/28807#issuecomment-643099104 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 #28808: [SPARK-31975][SQL] Throw user facing error when use WindowFunction directly

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

<    1   2   3   4   5   6   7   8   >