[GitHub] [spark] SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526789543 **[Test build #109977 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109977/te

[GitHub] [spark] viirya commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
viirya commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786398 cc @srowen This is an automated message f

[GitHub] [spark] AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786193 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786195 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
SparkQA removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778847 **[Test build #109976 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuil

[GitHub] [spark] AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786195 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786193 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
SparkQA commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526786106 **[Test build #109976 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526785562 Test FAILed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526785560 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526785560 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
SparkQA removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526773393 **[Test build #109971 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPul

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526785562 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526785474 **[Test build #109971 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReques

[GitHub] [spark] HeartSaVioR edited a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
HeartSaVioR edited a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526784751 It seems to be known flaky - https://issues.apache.org/jira/browse/SPARK-28770 - and not related to t

[GitHub] [spark] HeartSaVioR edited a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
HeartSaVioR edited a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526784751 It seems to be known flaky - https://issues.apache.org/jira/browse/SPARK-28770 - and not related to t

[GitHub] [spark] HeartSaVioR commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
HeartSaVioR commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526784751 It seems to be known flaky - https://issues.apache.org/jira/browse/SPARK-28770 - and not related to this pa

[GitHub] [spark] AmplabJenkins commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526784049 Merged build finished. Test PASSed. This is a

[GitHub] [spark] AmplabJenkins removed a comment on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526784052 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526784049 Merged build finished. Test PASSed. T

[GitHub] [spark] AmplabJenkins commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526784052 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] rdblue commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
rdblue commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526783648 @brkyvz, I updated this to address your comments. When you have a chance, please take another look. Thanks!

[GitHub] [spark] rdblue commented on a change in pull request #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
rdblue commented on a change in pull request #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#discussion_r319703305 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/v2/V2Sessi

[GitHub] [spark] SparkQA commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
SparkQA commented on issue #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#issuecomment-526783526 **[Test build #109980 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109980/testRep

[GitHub] [spark] rdblue commented on a change in pull request #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog

2019-08-30 Thread GitBox
rdblue commented on a change in pull request #25363: [SPARK-28628][SQL] Implement SupportsNamespaces in V2SessionCatalog URL: https://github.com/apache/spark/pull/25363#discussion_r319703205 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/v2/V2Sessi

[GitHub] [spark] maropu commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
maropu commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#discussion_r319702479 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec

[GitHub] [spark] SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526780824 **[Test build #109979 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequest

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526780537 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526780540 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526780540 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526780537 Merged build finished. Test PASSed.

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
dongjoon-hyun commented on a change in pull request #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#discussion_r319701011 ## File path: common/network-common/pom.xml ## @@ -8

[GitHub] [spark] viirya commented on a change in pull request #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
viirya commented on a change in pull request #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#discussion_r319700868 ## File path: common/network-common/pom.xml ## @@ -87,6 +87,

[GitHub] [spark] SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779903 **[Test build #109977 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109977/tes

[GitHub] [spark] SparkQA commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
SparkQA commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-526779902 **[Test build #109978 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109978/testReport)*

[GitHub] [spark] AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779597 Test PASSed. Refer to this link for build results (access rights to CI server needed): http

[GitHub] [spark] AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-526779641 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.ber

[GitHub] [spark] AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-526779641 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampla

[GitHub] [spark] AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-526779635 Merged build finished. Test PASSed. This is an aut

[GitHub] [spark] AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-526779635 Merged build finished. Test PASSed. This i

[GitHub] [spark] AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779595 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779597 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779595 Merged build finished. Test PASSed. This

[GitHub] [spark] maropu commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
maropu commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526779566 Sounds good to me, too. ---

[GitHub] [spark] maropu commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
maropu commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#discussion_r319700448 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/nullExpressi

[GitHub] [spark] dongjoon-hyun commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779034 Retest this please. This is an automated

[GitHub] [spark] dongjoon-hyun commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526779020 Ur, it's relevant failure. Although I'll trigger this once more, could you take a look? > org.apache.sp

[GitHub] [spark] AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778650 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778647 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
SparkQA commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778847 **[Test build #109976 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1099

[GitHub] [spark] AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526778481 Test FAILed. Refer to this link for build results (access rights to CI server needed): https

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778542 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.e

[GitHub] [spark] dongjoon-hyun edited a comment on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type

2019-08-30 Thread GitBox
dongjoon-hyun edited a comment on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526778602 https://issues.apache.org/jira/bro

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778539 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526778602 https://issues.apache.org/jira/browse/SPA

[GitHub] [spark] AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526778478 Merged build finished. Test FAILed. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778417 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.ed

[GitHub] [spark] AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778647 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526778650 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778539 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526778499 Yes. I think that might be a good choice.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526777219 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.e

[GitHub] [spark] SparkQA removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
SparkQA removed a comment on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526759731 **[Test build #109965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/10

[GitHub] [spark] AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526778481 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://ampla

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778414 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526778478 Merged build finished. Test FAILed. This

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778542 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenki

[GitHub] [spark] SparkQA removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
SparkQA removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526776876 **[Test build #109975 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109975/testReport)** for

[GitHub] [spark] SparkQA commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
SparkQA commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778408 **[Test build #109975 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109975/testReport)** for PR 2561

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778417 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkin

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526778414 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite

2019-08-30 Thread GitBox
SparkQA commented on issue #25629: [SPARK-28931][CORE][TESTS] Fix couple of bugs in FsHistoryProviderSuite URL: https://github.com/apache/spark/pull/25629#issuecomment-526778351 **[Test build #109965 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109965/te

[GitHub] [spark] dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526778085 Got it. So, Apache Spark 3.0.0 starts wit

[GitHub] [spark] dilipbiswal commented on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference

2019-08-30 Thread GitBox
dilipbiswal commented on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference URL: https://github.com/apache/spark/pull/25529#issuecomment-526777907 @gatorsmile Thank you for fixing the mistake and merging.

[GitHub] [spark] viirya opened a new pull request #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
viirya opened a new pull request #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639 ### What changes were proposed in this pull request? When Initializing factors in ALS, we should use `map

[GitHub] [spark] viirya commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors

2019-08-30 Thread GitBox
viirya commented on issue #25639: [SPARK-28933][ML] Reduce unnecessary shuffle in ALS when initializing factors URL: https://github.com/apache/spark/pull/25639#issuecomment-526777925 cc @felixcheung This is an automated mess

[GitHub] [spark] gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526777624 I think `spark` at first? When `pgSQL` mode

[GitHub] [spark] dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526777529 Yep. I agree. Sounds good to me. And, wha

[GitHub] [spark] gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526777381 I believe postgreSQL compatibility is a very

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526777215 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526777219 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenki

[GitHub] [spark] AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526777215 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] SparkQA commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
SparkQA commented on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526776876 **[Test build #109975 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109975/testReport)** for PR 25614

[GitHub] [spark] AmplabJenkins removed a comment on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775381 Can one of the admins verify this patch? This

[GitHub] [spark] SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
SparkQA commented on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775808 **[Test build #109973 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequest

[GitHub] [spark] SparkQA commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
SparkQA commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775801 **[Test build #109974 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109974/testReport)**

[GitHub] [spark] mukulmurthy commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
mukulmurthy commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775715 @HyukjinKwon , it is a little dangerous to add an API to directly break other module's custom listeners. But in some cas

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][BUILD] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775443 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type.

2019-08-30 Thread GitBox
gatorsmile commented on issue #25458: [SPARK-27931][SQL] Accept "true", "yes", "1", "false", "no", "0", and unique prefixes as input and trim input for the boolean data type. URL: https://github.com/apache/spark/pull/25458#issuecomment-526775542 @dongjoon-hyun @maropu @cloud-fan @HyukjinKw

[GitHub] [spark] AmplabJenkins removed a comment on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775440 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775443 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25638: [SPARK-28932][CORE] Add scala-library test dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775440 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775312 Can one of the admins verify this patch? This

[GitHub] [spark] AmplabJenkins commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775381 Can one of the admins verify this patch? This is an a

[GitHub] [spark] AmplabJenkins commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518#issuecomment-526775312 Can one of the admins verify this patch? This is an a

[GitHub] [spark] dongjoon-hyun commented on issue #25638: [SPARK-28932][CORE][test-hadoop3.2][test-maven] Add scala dependency to network-common module for JDK11

2019-08-30 Thread GitBox
dongjoon-hyun commented on issue #25638: [SPARK-28932][CORE][test-hadoop3.2][test-maven] Add scala dependency to network-common module for JDK11 URL: https://github.com/apache/spark/pull/25638#issuecomment-526775141 Retest this please. -

[GitHub] [spark] viirya commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-30 Thread GitBox
viirya commented on a change in pull request #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#discussion_r319696722 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526774756 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.ed

[GitHub] [spark] SparkQA removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
SparkQA removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526773409 **[Test build #109970 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109970/testReport)** for

[GitHub] [spark] AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25614: [WIP][SPARK-28887][K8S] Executor pod status fix URL: https://github.com/apache/spark/pull/25614#issuecomment-526774754 Merged build finished. Test FAILed. This is an automated

<    1   2   3   4   5   6   7   8   >