[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522278077 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522278078 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522278077 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522278078 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] maropu commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
maropu commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#discussion_r314962673 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scal

[GitHub] [spark] maropu commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
maropu commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#discussion_r314962673 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scal

[GitHub] [spark] SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522277856 **[Test build #109282 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/10928

[GitHub] [spark] maropu commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
maropu commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522277764 retest this please This is an automated mes

[GitHub] [spark] maropu commented on a change in pull request #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
maropu commented on a change in pull request #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#discussion_r314962435 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala ## @@ -156,

[GitHub] [spark] SparkQA commented on issue #25364: [SPARK-21067][DOC] Fix Thrift Server - CTAS fail with Unable to move source

2019-08-17 Thread GitBox
SparkQA commented on issue #25364: [SPARK-21067][DOC] Fix Thrift Server - CTAS fail with Unable to move source URL: https://github.com/apache/spark/pull/25364#issuecomment-522276526 **[Test build #4836 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4836/

[GitHub] [spark] AmplabJenkins removed a comment on issue #25364: [SPARK-21067][DOC] Fix Thrift Server - CTAS fail with Unable to move source

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25364: [SPARK-21067][DOC] Fix Thrift Server - CTAS fail with Unable to move source URL: https://github.com/apache/spark/pull/25364#issuecomment-518487093 Can one of the admins verify this patch? -

[GitHub] [spark] srowen commented on a change in pull request #25480: [SPARK-28763][SQL][TEST] Flaky Tests: SparkThriftServerProtocolVersionsSuite.HIVE_CLI_SERVICE_PROTOCOL_V1 get binary type

2019-08-17 Thread GitBox
srowen commented on a change in pull request #25480: [SPARK-28763][SQL][TEST] Flaky Tests: SparkThriftServerProtocolVersionsSuite.HIVE_CLI_SERVICE_PROTOCOL_V1 get binary type URL: https://github.com/apache/spark/pull/25480#discussion_r314959107 ## File path: sql/hive-thriftserver

[GitHub] [spark] srowen commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
srowen commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522269223 Aside from resolving the conflict (the exclusions are now already applied in master) an

[GitHub] [spark] srowen commented on issue #25423: [SPARK-28701][test-java11][k8s] adding java11 support for pull request builds

2019-08-17 Thread GitBox
srowen commented on issue #25423: [SPARK-28701][test-java11][k8s] adding java11 support for pull request builds URL: https://github.com/apache/spark/pull/25423#issuecomment-522269108 Er, or does this kind of depend on https://github.com/apache/spark/pull/25443 for Hive 2.x support? Maybe t

[GitHub] [spark] srowen commented on issue #25423: [SPARK-28701][test-java11][k8s] adding java11 support for pull request builds

2019-08-17 Thread GitBox
srowen commented on issue #25423: [SPARK-28701][test-java11][k8s] adding java11 support for pull request builds URL: https://github.com/apache/spark/pull/25423#issuecomment-522269068 OK, the current state of this change looks correct to me; it is what I would _expect_ the change looks like

[GitHub] [spark] AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522266948 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522266950 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.

[GitHub] [spark] AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522266950 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley

[GitHub] [spark] AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522266948 Merged build finished. Test PASSed. This is an automat

[GitHub] [spark] SparkQA removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252278 **[Test build #109280 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109280/testRepo

[GitHub] [spark] SparkQA commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
SparkQA commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522266832 **[Test build #109280 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109280/testReport)**

[GitHub] [spark] AmplabJenkins removed a comment on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262394 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins removed a comment on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262389 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
SparkQA commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262551 **[Test build #109281 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109281/testR

[GitHub] [spark] AmplabJenkins commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262394 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] [spark] AmplabJenkins commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262389 Merged build finished. Test PASSed. This is

[GitHub] [spark] sarutak commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
sarutak commented on issue #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484#issuecomment-522262338 CC: @dongjoon-hyun This is an automated message

[GitHub] [spark] sarutak opened a new pull request #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4

2019-08-17 Thread GitBox
sarutak opened a new pull request #25484: [SPARK-28647][WEBUI] Recover additional metric feature for branch-2.4 URL: https://github.com/apache/spark/pull/25484 This PR is for backporting SPARK-28647 to branch-2.4. The original PR removed `additional-metrics.js` but branch-2.4 still use i

[GitHub] [spark] dongjoon-hyun edited a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
dongjoon-hyun edited a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-522259383 Thank you for review and approval, @srowen . Merged to master/2.4/2.3. Also, cc @kiszk

[GitHub] [spark] dongjoon-hyun closed pull request #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
dongjoon-hyun closed pull request #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483 This is an automated message from the Apache Git Service. T

[GitHub] [spark] dongjoon-hyun commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
dongjoon-hyun commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-522259383 Thank you for review and approval, @srowen . Merged to master/2.4/2.3. -

[GitHub] [spark] dongjoon-hyun commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
dongjoon-hyun commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-522254932 Could you review this please, @srowen , @felixcheung , @HyukjinKwon ? -

[GitHub] [spark] dongjoon-hyun closed pull request #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
dongjoon-hyun closed pull request #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481 This is an automated message from the Apache Git Service.

[GitHub] [spark] dongjoon-hyun commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
dongjoon-hyun commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522254770 Then, thank you, @srowen , @HyukjinKwon , @wangyum . I'll merge this and follow this explicit testin

[GitHub] [spark] srowen commented on a change in pull request #22282: [SPARK-23539][SS] Add support for Kafka headers in Structured Streaming

2019-08-17 Thread GitBox
srowen commented on a change in pull request #22282: [SPARK-23539][SS] Add support for Kafka headers in Structured Streaming URL: https://github.com/apache/spark/pull/22282#discussion_r314953704 ## File path: external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/Kafk

[GitHub] [spark] srowen commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
srowen commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522254467 I'm OK with it then. This is an automated me

[GitHub] [spark] mickjermsurawong-stripe commented on a change in pull request #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
mickjermsurawong-stripe commented on a change in pull request #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#discussion_r314953309 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala ###

[GitHub] [spark] SparkQA commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
SparkQA commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252278 **[Test build #109280 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109280/testReport)** f

[GitHub] [spark] AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252151 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs

[GitHub] [spark] AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252149 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252151 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkele

[GitHub] [spark] AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25033: [SPARK-28224][SQL] Check overflow in decimal Sum aggregate URL: https://github.com/apache/spark/pull/25033#issuecomment-522252149 Merged build finished. Test PASSed. This is an automat

[GitHub] [spark] AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522250843 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522250842 Merged build finished. Test PASSed. This is an a

[GitHub] [spark] AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522250842 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522250843 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.be

[GitHub] [spark] SparkQA removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234173 **[Test build #109278 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109278/te

[GitHub] [spark] SparkQA commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
SparkQA commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522250716 **[Test build #109278 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109278/testRepor

[GitHub] [spark] HyukjinKwon commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
HyukjinKwon commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522244527 It was tested against JDK 11 at https://github.com/apache/spark/pull/25443#issuecomment-522235276 but ye

[GitHub] [spark] wangyum commented on a change in pull request #25480: [SPARK-28763][SQL][TEST] Flaky Tests: SparkThriftServerProtocolVersionsSuite.HIVE_CLI_SERVICE_PROTOCOL_V1 get binary type

2019-08-17 Thread GitBox
wangyum commented on a change in pull request #25480: [SPARK-28763][SQL][TEST] Flaky Tests: SparkThriftServerProtocolVersionsSuite.HIVE_CLI_SERVICE_PROTOCOL_V1 get binary type URL: https://github.com/apache/spark/pull/25480#discussion_r314949344 ## File path: sql/hive-thriftserve

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522243261 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522243258 Merged build finished. Test FAILed. -

[GitHub] [spark] SparkQA removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234172 **[Test build #109279 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuild

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522243258 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522243261 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522243153 **[Test build #109279 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1092

[GitHub] [spark] srowen commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
srowen commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522239829 Well the other solution is to explicitly include these additional libraries as a `runtime` dependency. Then t

[GitHub] [spark] xuanyuanking commented on a change in pull request #25445: [SPARK-28541][WEBUI] Document Storage page

2019-08-17 Thread GitBox
xuanyuanking commented on a change in pull request #25445: [SPARK-28541][WEBUI] Document Storage page URL: https://github.com/apache/spark/pull/25445#discussion_r314947902 ## File path: docs/web-ui.md ## @@ -45,6 +45,53 @@ The Storage tab displays the persisted RDDs and Da

[GitHub] [spark] MaxGekk commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
MaxGekk commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#discussion_r314947710 ## File path: sql/core/src/test/resources/sql-tests/inputs/pgSQL/timestamp.sql ## @@

[GitHub] [spark] AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237996 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237999 Test PASSed. Refer to this link for build results (access rights to CI server needed): h

[GitHub] [spark] AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237999 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://a

[GitHub] [spark] AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237996 Merged build finished. Test PASSed. T

[GitHub] [spark] SparkQA removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-58819 **[Test build #109276 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilde

[GitHub] [spark] AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237808 Test PASSed. Refer to this link for build results (access rights to CI server needed): h

[GitHub] [spark] SparkQA commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
SparkQA commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237823 **[Test build #109276 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/10927

[GitHub] [spark] AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237806 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237806 Merged build finished. Test PASSed. T

[GitHub] [spark] AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237808 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://a

[GitHub] [spark] SparkQA commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
SparkQA commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-522237634 **[Test build #109277 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/10927

[GitHub] [spark] SparkQA removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-59264 **[Test build #109277 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilde

[GitHub] [spark] AmplabJenkins removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522235428 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522235426 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522235428 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522235426 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522212566 **[Test build #109269 has started](https://amplab.cs.berkeley.edu/jenkins/job/

[GitHub] [spark] SparkQA commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins

2019-08-17 Thread GitBox
SparkQA commented on issue #25443: [WIP][SPARK-28723][test-hadoop3.2][test-maven] Test JDK 11 with Hadoop-3.2/Hive 2.3.6 on jenkins URL: https://github.com/apache/spark/pull/25443#issuecomment-522235276 **[Test build #109269 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPu

[GitHub] [spark] MaxGekk commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
MaxGekk commented on a change in pull request #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#discussion_r314946561 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.sca

[GitHub] [spark] AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234606 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amp

[GitHub] [spark] AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234604 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234606 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.b

[GitHub] [spark] AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234604 Merged build finished. Test PASSed. This is an a

[GitHub] [spark] SparkQA commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates

2019-08-17 Thread GitBox
SparkQA commented on issue #25410: [SPARK-28690][SQL] Add `date_part` function for timestamps/dates URL: https://github.com/apache/spark/pull/25410#issuecomment-522234173 **[Test build #109278 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109278/testReport

[GitHub] [spark] SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
SparkQA commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234172 **[Test build #109279 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/10927

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234056 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234056 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234055 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522234055 Merged build finished. Test PASSed. -

[GitHub] [spark] HyukjinKwon commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
HyukjinKwon commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-522233720 retest this please This is an automat

[GitHub] [spark] HyukjinKwon commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue

2019-08-17 Thread GitBox
HyukjinKwon commented on issue #25481: [SPARK-28765][BUILD] Add explict exclusions to avoid JDK11 dependency issue URL: https://github.com/apache/spark/pull/25481#issuecomment-59903 Given the test, this PR still looks good to me.

[GitHub] [spark] SparkQA removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
SparkQA removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-57106 **[Test build #109275 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder

[GitHub] [spark] AmplabJenkins removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-59575 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-59578 Test PASSed. Refer to this link for build results (access rights to CI server needed): ht

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-59521 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-59578 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://am

[GitHub] [spark] AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins removed a comment on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-59519 Merged build finished. Test FAILed. -

[GitHub] [spark] SparkQA commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
SparkQA commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-59544 **[Test build #109275 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109275

[GitHub] [spark] AmplabJenkins commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #25483: [SPARK-28766][R][DOC] Fix CRAN incoming feasibility warning on invalid URL URL: https://github.com/apache/spark/pull/25483#issuecomment-59575 Merged build finished. Test PASSed. Th

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-59521 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config

2019-08-17 Thread GitBox
AmplabJenkins commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-59519 Merged build finished. Test FAILed.

<    1   2   3   4   >