[GitHub] [spark] turboFei commented on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait

2020-03-25 Thread GitBox
turboFei commented on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait URL: https://github.com/apache/spark/pull/27943#issuecomment-603876706 thanks, have updated the description.

[GitHub] [spark] onursatici commented on issue #27695: [SPARK-30949][K8S][CORE] decouple requests and parallelism on kubernetes drivers

2020-03-25 Thread GitBox
onursatici commented on issue #27695: [SPARK-30949][K8S][CORE] decouple requests and parallelism on kubernetes drivers URL: https://github.com/apache/spark/pull/27695#issuecomment-603876593 @jiangxb1987 do you recommend to do that in this PR? I think changing the stand-alone driver core

[GitHub] [spark] SparkQA removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
SparkQA removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603733412 **[Test build #120332 has

[GitHub] [spark] AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603891883 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603891905 Test PASSed. Refer to this link for build results

[GitHub] [spark] AmplabJenkins removed a comment on issue #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables URL: https://github.com/apache/spark/pull/27902#issuecomment-603890506 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables URL: https://github.com/apache/spark/pull/27902#issuecomment-603890515 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603891905 Test PASSed. Refer to this link for build

[GitHub] [spark] AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603891883 Merged build finished. Test PASSed.

[GitHub] [spark] yaooqinn opened a new pull request #28020: [MINOR][BUILD][Unidoc] Pin the avro version in SBT

2020-03-25 Thread GitBox
yaooqinn opened a new pull request #28020: [MINOR][BUILD][Unidoc] Pin the avro version in SBT URL: https://github.com/apache/spark/pull/28020 ### What changes were proposed in this pull request? add arvo dep in SparkBuild ### Why are the changes needed? fix sbt

[GitHub] [spark] yaooqinn commented on issue #28020: [MINOR][BUILD][Unidoc] Pin the avro version in SBT

2020-03-25 Thread GitBox
yaooqinn commented on issue #28020: [MINOR][BUILD][Unidoc] Pin the avro version in SBT URL: https://github.com/apache/spark/pull/28020#issuecomment-603898818 cc @HyukjinKwon This is an automated message from the Apache Git

[GitHub] [spark] sarutak opened a new pull request #28021: [SPARK-31161][WEBUI][3.0] Refactor the on-click timeline action in streagming-page.js

2020-03-25 Thread GitBox
sarutak opened a new pull request #28021: [SPARK-31161][WEBUI][3.0] Refactor the on-click timeline action in streagming-page.js URL: https://github.com/apache/spark/pull/28021 ### What changes were proposed in this pull request? Refactor `streaming-page.js` by making

[GitHub] [spark] sarutak commented on issue #28021: [SPARK-31161][WEBUI][3.0] Refactor the on-click timeline action in streagming-page.js

2020-03-25 Thread GitBox
sarutak commented on issue #28021: [SPARK-31161][WEBUI][3.0] Refactor the on-click timeline action in streagming-page.js URL: https://github.com/apache/spark/pull/28021#issuecomment-603904923 cc: @gengliangwang This is an

[GitHub] [spark] AmplabJenkins commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector URL: https://github.com/apache/spark/pull/27978#issuecomment-603923438 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] SparkQA removed a comment on issue #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir

2020-03-25 Thread GitBox
SparkQA removed a comment on issue #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir URL: https://github.com/apache/spark/pull/27969#issuecomment-603769215 **[Test build #120340 has

[GitHub] [spark] Eric5553 commented on a change in pull request #27897: [SPARK-31113][SQL] Add SHOW VIEWS command

2020-03-25 Thread GitBox
Eric5553 commented on a change in pull request #27897: [SPARK-31113][SQL] Add SHOW VIEWS command URL: https://github.com/apache/spark/pull/27897#discussion_r397969899 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ## @@ -295,6

[GitHub] [spark] AmplabJenkins removed a comment on issue #27978: [SPARK-31127][ML] Implement abstract Selector

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27978: [SPARK-31127][ML] Implement abstract Selector URL: https://github.com/apache/spark/pull/27978#issuecomment-603923455 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27978: [SPARK-31127][ML] Implement abstract Selector

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27978: [SPARK-31127][ML] Implement abstract Selector URL: https://github.com/apache/spark/pull/27978#issuecomment-603923438 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector URL: https://github.com/apache/spark/pull/27978#issuecomment-603923455 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] huaxingao commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector

2020-03-25 Thread GitBox
huaxingao commented on issue #27978: [SPARK-31127][ML] Implement abstract Selector URL: https://github.com/apache/spark/pull/27978#issuecomment-603922940 retest this please This is an automated message from the Apache Git

[GitHub] [spark] huaxingao commented on issue #27994: [SPARK-31223][ML] Set seed in np.random to regenerate test data

2020-03-25 Thread GitBox
huaxingao commented on issue #27994: [SPARK-31223][ML] Set seed in np.random to regenerate test data URL: https://github.com/apache/spark/pull/27994#issuecomment-603923175 retest this please This is an automated message from

[GitHub] [spark] SparkQA commented on issue #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir

2020-03-25 Thread GitBox
SparkQA commented on issue #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir URL: https://github.com/apache/spark/pull/27969#issuecomment-603922945 **[Test build #120340 has

[GitHub] [spark] AmplabJenkins commented on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group URL: https://github.com/apache/spark/pull/27665#issuecomment-603812006 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group URL: https://github.com/apache/spark/pull/27665#issuecomment-603812006 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group URL: https://github.com/apache/spark/pull/27665#issuecomment-603811998 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27665: [SPARK-30623][Core] Spark external shuffle allow disable of separate event loop group URL: https://github.com/apache/spark/pull/27665#issuecomment-603811998 Merged build finished. Test PASSed.

[GitHub] [spark] LucaCanali commented on issue #26953: [SPARK-30306][CORE][PYTHON] Instrument Python UDF execution time and throughput metrics using Spark Metrics system

2020-03-25 Thread GitBox
LucaCanali commented on issue #26953: [SPARK-30306][CORE][PYTHON] Instrument Python UDF execution time and throughput metrics using Spark Metrics system URL: https://github.com/apache/spark/pull/26953#issuecomment-603811972 It looks like a case of flaky test. Can we test this again please?

[GitHub] [spark] AmplabJenkins commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603815570 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603815564 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] SparkQA removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
SparkQA removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603807279 **[Test build #120348 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/120348/testReport)** for

[GitHub] [spark] AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603815564 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] SparkQA commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
SparkQA commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603815460 **[Test build #120348 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/120348/testReport)** for PR

[GitHub] [spark] gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector

2020-03-25 Thread GitBox
gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector URL: https://github.com/apache/spark/pull/28019#issuecomment-603822330 Seems unrelated. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector URL: https://github.com/apache/spark/pull/28019#issuecomment-603821664 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector

2020-03-25 Thread GitBox
gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector URL: https://github.com/apache/spark/pull/28019#issuecomment-603822376 retest this please This is an automated

[GitHub] [spark] HyukjinKwon commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
HyukjinKwon commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603829134 Hm? This is an automated message from the Apache Git Service. To

[GitHub] [spark] SparkQA removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
SparkQA removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603819831 **[Test build #120350 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/120350/testReport)** for

[GitHub] [spark] SparkQA removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test

2020-03-25 Thread GitBox
SparkQA removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test URL: https://github.com/apache/spark/pull/28005#issuecomment-603758312 **[Test build #120337 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603828654 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v)) URL: https://github.com/apache/spark/pull/27803#issuecomment-603829329 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v)) URL: https://github.com/apache/spark/pull/27803#issuecomment-603829337 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v))

2020-03-25 Thread GitBox
SparkQA commented on issue #27803: [SPARK-31049][SQL] Support nested adjacent generators, e.g., explode(explode(v)) URL: https://github.com/apache/spark/pull/27803#issuecomment-603829274 **[Test build #120352 has

[GitHub] [spark] AmplabJenkins commented on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test URL: https://github.com/apache/spark/pull/28005#issuecomment-603829537 Test FAILed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test URL: https://github.com/apache/spark/pull/28005#issuecomment-603829524 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#issuecomment-603835003 Merged build finished. Test FAILed. This is

[GitHub] [spark] HyukjinKwon commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
HyukjinKwon commented on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603835425 Seems like different avro is being picked up during unidoc for some reasons: ``` [info] Main Java API documentation to

[GitHub] [spark] AmplabJenkins removed a comment on issue #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#issuecomment-603835013 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-603835149 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test URL: https://github.com/apache/spark/pull/28005#issuecomment-603843379 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603843416 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] yaooqinn commented on a change in pull request #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir

2020-03-25 Thread GitBox
yaooqinn commented on a change in pull request #27969: [SPARK-31170][SQL][test-hive1.2] Spark SQL Cli should respect hive-site.xml and spark.sql.warehouse.dir URL: https://github.com/apache/spark/pull/27969#discussion_r397858453 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28017: [MINOR][DOCS] Fix some links for python api doc URL: https://github.com/apache/spark/pull/28017#issuecomment-603843429 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28005: [SPARK-31231][BUILD][FOLLOW-UP] Set the upper bound (before 46.1.0) for setuptools in pip package test URL: https://github.com/apache/spark/pull/28005#issuecomment-603843371 Merged build finished. Test PASSed.

[GitHub] [spark] gaborgsomogyi commented on a change in pull request #27649: [SPARK-30900][SS] FileStreamSource: Avoid reading compact metadata log twice if the query restarts from compact batch

2020-03-25 Thread GitBox
gaborgsomogyi commented on a change in pull request #27649: [SPARK-30900][SS] FileStreamSource: Avoid reading compact metadata log twice if the query restarts from compact batch URL: https://github.com/apache/spark/pull/27649#discussion_r397858474 ## File path:

[GitHub] [spark] tgravescs commented on issue #27998: [SPARK-31219][YARN] Enable closeIdleConnections in YarnShuffleService

2020-03-25 Thread GitBox
tgravescs commented on issue #27998: [SPARK-31219][YARN] Enable closeIdleConnections in YarnShuffleService URL: https://github.com/apache/spark/pull/27998#issuecomment-603911283 so to clarify, you are saying that if connections timeout (from spark.shuffle.io.connectionTimeout or

[GitHub] [spark] dongjoon-hyun closed pull request #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
dongjoon-hyun closed pull request #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013 This is an automated message from

[GitHub] [spark] Eric5553 commented on a change in pull request #27897: [SPARK-31113][SQL] Add SHOW VIEWS command

2020-03-25 Thread GitBox
Eric5553 commented on a change in pull request #27897: [SPARK-31113][SQL] Add SHOW VIEWS command URL: https://github.com/apache/spark/pull/27897#discussion_r397975602 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveSessionCatalog.scala

[GitHub] [spark] gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector

2020-03-25 Thread GitBox
gaborgsomogyi commented on issue #28019: [SPARK-31021][SQL] Support MariaDB Kerberos login in JDBC connector URL: https://github.com/apache/spark/pull/28019#issuecomment-603935012 Filed https://issues.apache.org/jira/browse/SPARK-31252

[GitHub] [spark] AngersZhuuuu commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-25 Thread GitBox
AngersZh commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603657534 cc @HyukjinKwon Split to two pr, thanks

[GitHub] [spark] cloud-fan commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types URL: https://github.com/apache/spark/pull/26942#discussion_r397625350 ## File path:

[GitHub] [spark] yaooqinn commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types

2020-03-25 Thread GitBox
yaooqinn commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types URL: https://github.com/apache/spark/pull/26942#discussion_r397625334 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait URL: https://github.com/apache/spark/pull/27943#issuecomment-603657152 Merged build finished. Test PASSed.

[GitHub] [spark] cloud-fan commented on a change in pull request #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables URL: https://github.com/apache/spark/pull/27902#discussion_r397624892 ## File path: docs/sql-migration-guide.md ## @@ -111,6 +111,8 @@ license: | -

[GitHub] [spark] cloud-fan commented on a change in pull request #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #27902: [SPARK-31147][SQL] Forbid CHAR type in non-Hive-Serde tables URL: https://github.com/apache/spark/pull/27902#discussion_r397625011 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types URL: https://github.com/apache/spark/pull/26942#discussion_r397625437 ## File path:

[GitHub] [spark] yaooqinn commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types

2020-03-25 Thread GitBox
yaooqinn commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types URL: https://github.com/apache/spark/pull/26942#discussion_r397625503 ## File path:

[GitHub] [spark] AngersZhuuuu opened a new pull request #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-25 Thread GitBox
AngersZh opened a new pull request #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011 ### What changes were proposed in this pull request? Fix ExecutorMetricsPoller class comment comment

[GitHub] [spark] cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#discussion_r397628498 ## File path: docs/sql-ref-syntax-aux-conf-mgmt-reset.md ## @@ -20,7 +20,7 @@

[GitHub] [spark] cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#discussion_r397628498 ## File path: docs/sql-ref-syntax-aux-conf-mgmt-reset.md ## @@ -20,7 +20,7 @@

[GitHub] [spark] dongjoon-hyun opened a new pull request #28015: [SPARK-31244][K8S][TEST] Use Minio instead of Ceph in K8S DepsTestsSuite

2020-03-25 Thread GitBox
dongjoon-hyun opened a new pull request #28015: [SPARK-31244][K8S][TEST] Use Minio instead of Ceph in K8S DepsTestsSuite URL: https://github.com/apache/spark/pull/28015 ### What changes were proposed in this pull request? ### Why are the changes needed? ###

[GitHub] [spark] Ngone51 commented on a change in pull request #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
Ngone51 commented on a change in pull request #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#discussion_r397628164 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013#issuecomment-603659415 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark URL: https://github.com/apache/spark/pull/28012#issuecomment-603659402 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
SparkQA commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661291 **[Test build #120314 has

[GitHub] [spark] SparkQA commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark

2020-03-25 Thread GitBox
SparkQA commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark URL: https://github.com/apache/spark/pull/28012#issuecomment-603661341 **[Test build #120316 has

[GitHub] [spark] SparkQA commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
SparkQA commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013#issuecomment-603661314 **[Test build #120315 has

[GitHub] [spark] yaooqinn commented on a change in pull request #27956: [SPARK-31189][SQL][DOCS] Fix errors and missing parts for datetime pattern document

2020-03-25 Thread GitBox
yaooqinn commented on a change in pull request #27956: [SPARK-31189][SQL][DOCS] Fix errors and missing parts for datetime pattern document URL: https://github.com/apache/spark/pull/27956#discussion_r397628847 ## File path: python/pyspark/sql/streaming.py ## @@ -491,6

[GitHub] [spark] cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#discussion_r397628721 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala

[GitHub] [spark] AmplabJenkins removed a comment on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013#issuecomment-603659421 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27991: [SPARK-31227][SQL] Non-nullable null type in complex types should not coerce to nullable type

2020-03-25 Thread GitBox
HyukjinKwon commented on a change in pull request #27991: [SPARK-31227][SQL] Non-nullable null type in complex types should not coerce to nullable type URL: https://github.com/apache/spark/pull/27991#discussion_r397628911 ## File path:

[GitHub] [spark] Ngone51 commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
Ngone51 commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661646 cc @cloud-fan @jose-torres

[GitHub] [spark] AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661631 Merged build finished. Test PASSed.

[GitHub] [spark] cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #28003: [SPARK-31234][SQL] ResetCommand should reset config to sc.conf only URL: https://github.com/apache/spark/pull/28003#discussion_r397628937 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661638 Test PASSed. Refer to this link for build

[GitHub] [spark] AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661631 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark

2020-03-25 Thread GitBox
AmplabJenkins removed a comment on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark URL: https://github.com/apache/spark/pull/28012#issuecomment-603659394 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #28015: [SPARK-31244][K8S][TEST] Use Minio instead of Ceph in K8S DepsTestsSuite

2020-03-25 Thread GitBox
SparkQA commented on issue #28015: [SPARK-31244][K8S][TEST] Use Minio instead of Ceph in K8S DepsTestsSuite URL: https://github.com/apache/spark/pull/28015#issuecomment-603661316 **[Test build #120313 has

[GitHub] [spark] AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28014: [SPARK-31242][SQL][TEST] mergeSparkConf in WithTestConf should also respect spark.sql.legacy.sessionInitWithConfigDefaults URL: https://github.com/apache/spark/pull/28014#issuecomment-603661638 Test PASSed. Refer to this link for build results

[GitHub] [spark] manuzhang commented on issue #27998: [SPARK-31219][YARN] Enable closeIdleConnections in YarnShuffleService

2020-03-25 Thread GitBox
manuzhang commented on issue #27998: [SPARK-31219][YARN] Enable closeIdleConnections in YarnShuffleService URL: https://github.com/apache/spark/pull/27998#issuecomment-603665180 cc @vanzin @xuanyuanking @tgravescs who have worked on this module

[GitHub] [spark] Ngone51 commented on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait

2020-03-25 Thread GitBox
Ngone51 commented on issue #27943: [SPARK-31179] Fast fail the connection while last connection failed in the last retry IO wait URL: https://github.com/apache/spark/pull/27943#issuecomment-603664381 LGTM This is an

[GitHub] [spark] AmplabJenkins commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603658927 Can one of the admins verify this patch?

[GitHub] [spark] AngersZhuuuu commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-25 Thread GitBox
AngersZh commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603663192 > No~ @AngersZh. I meant to make one minor PR to fix many typos instead of making one PR

[GitHub] [spark] Ngone51 commented on a change in pull request #27967: [SPARK-31201][SQL] Add an individual config for skewed partition threshold

2020-03-25 Thread GitBox
Ngone51 commented on a change in pull request #27967: [SPARK-31201][SQL] Add an individual config for skewed partition threshold URL: https://github.com/apache/spark/pull/27967#discussion_r397630336 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013#issuecomment-603659421 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] cloud-fan commented on a change in pull request #27967: [SPARK-31201][SQL] Add an individual config for skewed partition threshold

2020-03-25 Thread GitBox
cloud-fan commented on a change in pull request #27967: [SPARK-31201][SQL] Add an individual config for skewed partition threshold URL: https://github.com/apache/spark/pull/27967#discussion_r397626940 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark URL: https://github.com/apache/spark/pull/28012#issuecomment-603659402 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28012: [SPARK-31243][ML][PySpark] Add ANOVATest and FValueTest to PySpark URL: https://github.com/apache/spark/pull/28012#issuecomment-603659394 Merged build finished. Test PASSed. This is

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types

2020-03-25 Thread GitBox
HyukjinKwon commented on a change in pull request #26942: [SPARK-30301][SQL] Fix wrong results when datetimes as fields of complex types URL: https://github.com/apache/spark/pull/26942#discussion_r397626946 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28011: [SPARK-26329][CORE][FOLLOWUP] Fix ExecutorMetricsPoller class comment miss parameter URL: https://github.com/apache/spark/pull/28011#issuecomment-603659289 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints

2020-03-25 Thread GitBox
AmplabJenkins commented on issue #28013: [SPARK-25121][SQL][FOLLOWUP] Add more unit tests for multi-part identifiers in join strategy hints URL: https://github.com/apache/spark/pull/28013#issuecomment-603659415 Merged build finished. Test PASSed.

<    4   5   6   7   8   9   10   11   12   13   >