[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

[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

[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

[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

[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

[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

[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):

[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.

[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):

[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:

[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

[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:

[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:

[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

[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

[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 ## @@

[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

[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

[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

[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):

[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):

[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):

[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

[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

[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):

[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:

[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? >

[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

[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):

[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):

[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

[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] 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

[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] 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):

[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):

[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 #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):

[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

[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):

[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):

[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

[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):

[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

[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

[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

[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

[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,

[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

[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):

[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

[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?

[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

[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

[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

[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?

[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

[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

[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:

[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):

[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

[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-526774754 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-526774742 **[Test build #109970 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109970/testReport)** for PR

[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-526774756 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] mukulmurthy opened a new pull request #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners()

2019-08-30 Thread GitBox
mukulmurthy opened a new pull request #25518: [SPARK-26046][SS] Add StreamingQueryManager.listListeners() URL: https://github.com/apache/spark/pull/25518 ### What changes were proposed in this pull request? Add a removeAllListeners() method to StreamingQueryManager that removes all

[GitHub] [spark] SparkQA commented on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API.

2019-08-30 Thread GitBox
SparkQA commented on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API. URL: https://github.com/apache/spark/pull/25304#issuecomment-526774581 **[Test build #109972 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API.

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API. URL: https://github.com/apache/spark/pull/25304#issuecomment-526774311 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API.

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API. URL: https://github.com/apache/spark/pull/25304#issuecomment-526774309 Merged build finished. Test PASSed.

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

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference URL: https://github.com/apache/spark/pull/25529#issuecomment-526774289 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference URL: https://github.com/apache/spark/pull/25529#issuecomment-526774289 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference

2019-08-30 Thread GitBox
AmplabJenkins removed a comment on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference URL: https://github.com/apache/spark/pull/25529#issuecomment-526774284 Build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API.

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25304: [SPARK-28570][CORE][SHUFFLE] Make UnsafeShuffleWriter use the new API. URL: https://github.com/apache/spark/pull/25304#issuecomment-526774309 Merged build finished. Test PASSed. This

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

2019-08-30 Thread GitBox
AmplabJenkins commented on issue #25529: [SPARK-28804][DOCS][SQL] Document DESCRIBE QUERY in SQL Reference URL: https://github.com/apache/spark/pull/25529#issuecomment-526774284 Build finished. Test PASSed. This is an

<    1   2   3   4   5   6   7   8   >