[GitHub] [spark] AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616182916 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616182906 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616182916 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] SparkQA removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
SparkQA removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145669 **[Test build #121476 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260#issuecomment-616182271 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260#issuecomment-616182271 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
SparkQA commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616182388 **[Test build #121476 has

[GitHub] [spark] AmplabJenkins commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260#issuecomment-616182265 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260#issuecomment-616182265 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
SparkQA commented on issue #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260#issuecomment-616182040 **[Test build #121480 has

[GitHub] [spark] Ngone51 opened a new pull request #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl

2020-04-19 Thread GitBox
Ngone51 opened a new pull request #28260: [SPARK-31487][CORE] Move slots check of barrier job from DAGScheduler to TaskSchedulerImpl URL: https://github.com/apache/spark/pull/28260 ### What changes were proposed in this pull request? Move slots check of barrier job from

[GitHub] [spark] AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616179936 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616179932 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616179936 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616179932 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
SparkQA removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144063 **[Test build #121475 has

[GitHub] [spark] SparkQA commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
SparkQA commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616179280 **[Test build #121475 has

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite

2020-04-19 Thread GitBox
dongjoon-hyun commented on a change in pull request #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite URL: https://github.com/apache/spark/pull/28173#discussion_r410939829 ## File path:

[GitHub] [spark] dongjoon-hyun removed a comment on issue #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite

2020-04-19 Thread GitBox
dongjoon-hyun removed a comment on issue #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite URL: https://github.com/apache/spark/pull/28173#issuecomment-616174002 @sririshindra . Could you address @maropu 's last comment. -

[GitHub] [spark] dongjoon-hyun commented on issue #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite

2020-04-19 Thread GitBox
dongjoon-hyun commented on issue #28173: [SPARK-31389][SQL][TESTS] Add codegen-off test coverage for some tests in SQLMetricsSuite URL: https://github.com/apache/spark/pull/28173#issuecomment-616174002 @sririshindra . Could you address @maropu 's last comment. -

[GitHub] [spark] dongjoon-hyun commented on issue #28173: [SPARK-31389][SQL][TESTS] Ensure all tests in SQLMetricsSuite run with both codegen on and off

2020-04-19 Thread GitBox
dongjoon-hyun commented on issue #28173: [SPARK-31389][SQL][TESTS] Ensure all tests in SQLMetricsSuite run with both codegen on and off URL: https://github.com/apache/spark/pull/28173#issuecomment-616173410 Thank you for confirming the intention of the PR, @sririshindra . I'll update the

[GitHub] [spark] dongjoon-hyun closed pull request #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
dongjoon-hyun closed pull request #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255 This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259#issuecomment-616170669 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259#issuecomment-616170667 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259#issuecomment-616170669 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259#issuecomment-616170667 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
SparkQA commented on issue #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259#issuecomment-616170383 **[Test build #121479 has

[GitHub] [spark] MaxGekk opened a new pull request #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown

2020-04-19 Thread GitBox
MaxGekk opened a new pull request #28259: [SPARK-31488][SQL] Support `java.time.LocalDate` in Parquet filter pushdown URL: https://github.com/apache/spark/pull/28259 ### What changes were proposed in this pull request? 1. Modified `ParquetFilters.valueCanMakeFilterOn()` to accept

[GitHub] [spark] dongjoon-hyun commented on issue #28235: [SPARK-31464][BUILD][SS] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
dongjoon-hyun commented on issue #28235: [SPARK-31464][BUILD][SS] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616169185 Hi, @viirya , @HyukjinKwon , @maropu . Could you review this PR, please? - For me, the test coverage looks good to me

[GitHub] [spark] dongjoon-hyun commented on issue #28235: [SPARK-31464][BUILD][SS] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
dongjoon-hyun commented on issue #28235: [SPARK-31464][BUILD][SS] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616167536 As I wrote in the PR description, the following is the current status. - [x] SBT

[GitHub] [spark] AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616166383 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616166383 Test PASSed. Refer to this link for build results

[GitHub] [spark] AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616166378 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616166378 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
SparkQA removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131231 **[Test build #121474 has

[GitHub] [spark] SparkQA commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
SparkQA commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616165894 **[Test build #121474 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258#issuecomment-616165199 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258#issuecomment-616165432 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258#issuecomment-616165199 Can one of the admins verify this patch?

[GitHub] [spark] akshatb1 closed pull request #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
akshatb1 closed pull request #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258 This is an automated message

[GitHub] [spark] akshatb1 opened a new pull request #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
akshatb1 opened a new pull request #28258: [SPARK-31486] spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258 ### What changes were proposed in this pull request? These changes implement

[GitHub] [spark] AmplabJenkins commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function URL: https://github.com/apache/spark/pull/28248#issuecomment-616163972 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function URL: https://github.com/apache/spark/pull/28248#issuecomment-616163975 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins removed a comment on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function URL: https://github.com/apache/spark/pull/28248#issuecomment-616163972 Merged build finished. Test PASSed.

[GitHub] [spark] akshatb1 opened a new pull request #28258: spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode

2020-04-19 Thread GitBox
akshatb1 opened a new pull request #28258: spark.submit.waitForCompletion flag to control spark-submit exit in Standalone Cluster Mode URL: https://github.com/apache/spark/pull/28258 ### What changes were proposed in this pull request? These changes implement an application

[GitHub] [spark] AmplabJenkins commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function URL: https://github.com/apache/spark/pull/28248#issuecomment-616163975 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] SparkQA commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function

2020-04-19 Thread GitBox
SparkQA commented on issue #28248: [SPARK-31474][SQL] Consistency between dayofweek/dow in extract exprsession and dayofweek function URL: https://github.com/apache/spark/pull/28248#issuecomment-616163605 **[Test build #121478 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616156339 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616156344 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616156339 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616156344 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
SparkQA commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616156014 **[Test build #121477 has

[GitHub] [spark] turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-61615 Have refactor the indents and double check it. cc @maropu @dongjoon-hyun

[GitHub] [spark] maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference URL: https://github.com/apache/spark/pull/28237#discussion_r410916442 ## File path: docs/sql-ref-literals.md ## @@ -0,0 +1,505 @@ +--- +layout: global +title:

[GitHub] [spark] maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference URL: https://github.com/apache/spark/pull/28237#discussion_r410916489 ## File path: docs/sql-ref-literals.md ## @@ -0,0 +1,505 @@ +--- +layout: global +title:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145952 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145960 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145960 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145952 Merged build finished. Test PASSed.

[GitHub] [spark] maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28237: [SPARK-31465][SQL][DOCS] Document Literal in SQL Reference URL: https://github.com/apache/spark/pull/28237#discussion_r410916091 ## File path: docs/sql-ref-literals.md ## @@ -0,0 +1,385 @@ +--- +layout: global +title:

[GitHub] [spark] SparkQA commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
SparkQA commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616145669 **[Test build #121476 has

[GitHub] [spark] Ngone51 commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

2020-04-19 Thread GitBox
Ngone51 commented on issue #28245: [SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check URL: https://github.com/apache/spark/pull/28245#issuecomment-616144694 retest this please.

[GitHub] [spark] AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144342 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] Ngone51 commented on a change in pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
Ngone51 commented on a change in pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#discussion_r410914798 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144342 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] Ngone51 commented on a change in pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
Ngone51 commented on a change in pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#discussion_r410914630 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144339 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144339 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
SparkQA commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144063 **[Test build #121475 has

[GitHub] [spark] Ngone51 commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
Ngone51 commented on issue #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257#issuecomment-616144009 ping @jiangxb1987 @cloud-fan please take a look, thanks!

[GitHub] [spark] Ngone51 opened a new pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched

2020-04-19 Thread GitBox
Ngone51 opened a new pull request #28257: [SPARK-31485][CORE] Avoid application hang if only partial barrier tasks launched URL: https://github.com/apache/spark/pull/28257 ### What changes were proposed in this pull request? Use `dagScheduler.taskSetFailed` to abort a

[GitHub] [spark] AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616140708 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616140714 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616140708 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616140714 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
SparkQA removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616087130 **[Test build #121473 has

[GitHub] [spark] SparkQA commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
SparkQA commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616140284 **[Test build #121473 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616139157 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616139153 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616139153 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616139157 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
SparkQA commented on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616138840 **[Test build #121472 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121472/testReport)**

[GitHub] [spark] SparkQA removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0

2020-04-19 Thread GitBox
SparkQA removed a comment on issue #28235: [SPARK-31464][BUILD][SS][test-maven] Upgrade Kafka to 2.5.0 URL: https://github.com/apache/spark/pull/28235#issuecomment-616062175 **[Test build #121472 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131440 Test PASSed. Refer to this link for build results

[GitHub] [spark] AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131438 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131438 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131440 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] SparkQA commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
SparkQA commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616131231 **[Test build #121474 has

[GitHub] [spark] HyukjinKwon commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception

2020-04-19 Thread GitBox
HyukjinKwon commented on issue #28255: [SPARK-31484][Core] Add stage attempt number to temp checkpoint filename to avoid file already existing exception URL: https://github.com/apache/spark/pull/28255#issuecomment-616130191 retest this please

[GitHub] [spark] turboFei edited a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
turboFei edited a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616096011 thanks, will fix all indents related with `""" """`.

[GitHub] [spark] turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616096011 thanks, would fix all indents related with `""" """`.

[GitHub] [spark] maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#discussion_r410864812 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#discussion_r410866961 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#discussion_r410865751 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#discussion_r410865751 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on a change in pull request #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#discussion_r410864812 ## File path:

[GitHub] [spark] maropu commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616088399 Ur, I see. cc: @dongjoon-hyun

[GitHub] [spark] turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
turboFei commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616087763 Oh, I mean that if we createOrReplaceTempView and does not clean up it after this test. If

[GitHub] [spark] AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins removed a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616087511 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
AmplabJenkins commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616087511 Merged build finished. Test PASSed.

<    1   2   3   4   5   >