[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569214561 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569214554 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569214561 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569214554 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569187757 **[Test build #115841 has

[GitHub] [spark] SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569214248 **[Test build #115841 has

[GitHub] [spark] cloud-fan commented on issue #26589: [SPARK-29947][SQL] Improve ResolveRelations performance

2019-12-26 Thread GitBox
cloud-fan commented on issue #26589: [SPARK-29947][SQL] Improve ResolveRelations performance URL: https://github.com/apache/spark/pull/26589#issuecomment-569213740 shall we put the map in `AnalysisContext`? Then we can make sure we only resolve the table once in a query, including tables

[GitHub] [spark] cloud-fan commented on issue #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty

2019-12-26 Thread GitBox
cloud-fan commented on issue #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty URL: https://github.com/apache/spark/pull/26477#issuecomment-569213416 I agree that NULL makes more sense in this case, but returning the original string is not a bad

[GitHub] [spark] AmplabJenkins removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569212477 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569212472 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569212472 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569178230 **[Test build #115835 has

[GitHub] [spark] AmplabJenkins commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569212477 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] HeartSaVioR commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite

2019-12-26 Thread GitBox
HeartSaVioR commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite URL: https://github.com/apache/spark/pull/27001#issuecomment-569212370 Thanks all for reviewing and merging!

[GitHub] [spark] SparkQA commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1

2019-12-26 Thread GitBox
SparkQA commented on issue #27009: [WIP][SPARK-29250][BUILD][test-hadoop3.2][test-maven] Upgrade to Hadoop 3.2.1 URL: https://github.com/apache/spark/pull/27009#issuecomment-569212389 **[Test build #115835 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569212050 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569212047 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569212050 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty URL: https://github.com/apache/spark/pull/26477#discussion_r361601897 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty URL: https://github.com/apache/spark/pull/26477#discussion_r361601897 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569212047 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569176104 **[Test build #115834 has

[GitHub] [spark] SparkQA commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
SparkQA commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569211719 **[Test build #115834 has

[GitHub] [spark] cloud-fan closed pull request #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite

2019-12-26 Thread GitBox
cloud-fan closed pull request #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite URL: https://github.com/apache/spark/pull/27001 This

[GitHub] [spark] cloud-fan commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite

2019-12-26 Thread GitBox
cloud-fan commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite URL: https://github.com/apache/spark/pull/27001#issuecomment-569211058 thanks, merging to master!

[GitHub] [spark] AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569210443 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#issuecomment-569210474 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569210437 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#issuecomment-569210478 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#issuecomment-569210478 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#issuecomment-569210474 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569210443 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569210437 Merged build finished. Test PASSed.

[GitHub] [spark] Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361600186 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] SparkQA commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
SparkQA commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569210248 **[Test build #115846 has

[GitHub] [spark] SparkQA commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
SparkQA commented on issue #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#issuecomment-569210266 **[Test build #115847 has

[GitHub] [spark] HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r36155 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] AmplabJenkins removed a comment on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205 URL: https://github.com/apache/spark/pull/27022#issuecomment-569208787 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205 URL: https://github.com/apache/spark/pull/27022#issuecomment-569209034 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205 URL: https://github.com/apache/spark/pull/27022#issuecomment-569208787 Can one of the admins verify this patch?

[GitHub] [spark] Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361598648 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361597144 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361598001 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] spektom opened a new pull request #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205

2019-12-26 Thread GitBox
spektom opened a new pull request #27022: [SPARK-28415][DSTREAMS] Add messageHandler to Kafka 10 direct stream API #25205 URL: https://github.com/apache/spark/pull/27022 ## What changes were proposed in this pull request? This patch introduces messageHandler parameter that can be

[GitHub] [spark] Ngone51 commented on issue #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint

2019-12-26 Thread GitBox
Ngone51 commented on issue #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint URL: https://github.com/apache/spark/pull/27012#issuecomment-569207181 Thanks @jiangxb1987 @cloud-fan

[GitHub] [spark] HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
HeartSaVioR commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361597144 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] AbhishekNew commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty

2019-12-26 Thread GitBox
AbhishekNew commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty URL: https://github.com/apache/spark/pull/26477#discussion_r361597143 ## File path:

[GitHub] [spark] yaooqinn commented on a change in pull request #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
yaooqinn commented on a change in pull request #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#discussion_r361596662 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569206218 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569206213 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569206213 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569206218 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] ulysses-you commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
ulysses-you commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#discussion_r361596435 ## File path:

[GitHub] [spark] ulysses-you commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
ulysses-you commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#discussion_r361596435 ## File path:

[GitHub] [spark] SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569205933 **[Test build #115845 has

[GitHub] [spark] yaooqinn commented on a change in pull request #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
yaooqinn commented on a change in pull request #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#discussion_r361595611 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569204645 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569204641 Merged build finished. Test PASSed.

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26589: [SPARK-29947][SQL] Improve ResolveRelations performance

2019-12-26 Thread GitBox
HyukjinKwon commented on a change in pull request #26589: [SPARK-29947][SQL] Improve ResolveRelations performance URL: https://github.com/apache/spark/pull/26589#discussion_r361595210 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala

[GitHub] [spark] AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569204641 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569204645 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569173791 **[Test build #115833 has

[GitHub] [spark] SparkQA commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found

2019-12-26 Thread GitBox
SparkQA commented on issue #25399: [SPARK-28670][SQL] create function should thrown Exception if the resource is not found URL: https://github.com/apache/spark/pull/25399#issuecomment-569204340 **[Test build #115833 has

[GitHub] [spark] cloud-fan commented on issue #24495: [SPARK-27604][SQL] Add filter reduction

2019-12-26 Thread GitBox
cloud-fan commented on issue #24495: [SPARK-27604][SQL] Add filter reduction URL: https://github.com/apache/spark/pull/24495#issuecomment-569203624 can we enrich the PR description with how it is implemented? e.g. the overall design, the algorithm that is used.

[GitHub] [spark] Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361594137 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage

2019-12-26 Thread GitBox
Ngone51 commented on a change in pull request #27010: [SPARK-30313][CORE] Ensure EndpointRef is available MasterWebUI/WorkerPage URL: https://github.com/apache/spark/pull/27010#discussion_r361594038 ## File path: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala

[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569203034 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] cloud-fan commented on issue #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint

2019-12-26 Thread GitBox
cloud-fan commented on issue #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint URL: https://github.com/apache/spark/pull/27012#issuecomment-569203115 thanks, merging to master!

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569203034 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569203032 Merged build finished. Test PASSed.

[GitHub] [spark] HyukjinKwon commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty

2019-12-26 Thread GitBox
HyukjinKwon commented on a change in pull request #26477: [SPARK-29776][SQL] rpad and lpad should return NULL when padstring parameter is empty URL: https://github.com/apache/spark/pull/26477#discussion_r361593735 ## File path:

[GitHub] [spark] cloud-fan closed pull request #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint

2019-12-26 Thread GitBox
cloud-fan closed pull request #27012: [SPARK-30355][CORE] Unify isExecutorActive between CoarseGrainedSchedulerBackend and DriverEndpoint URL: https://github.com/apache/spark/pull/27012 This is an automated message from the

[GitHub] [spark] AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569202946 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569202946 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569203032 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569202941 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569202941 Merged build finished. Test PASSed. This

[GitHub] [spark] SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569202745 **[Test build #115836 has

[GitHub] [spark] SparkQA removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569179292 **[Test build #115836 has

[GitHub] [spark] SparkQA removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
SparkQA removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569170150 **[Test build #115832 has

[GitHub] [spark] SparkQA commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
SparkQA commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569202596 **[Test build #115832 has

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#discussion_r361593116 ## File path:

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#discussion_r361593116 ## File path:

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
HyukjinKwon commented on a change in pull request #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#discussion_r361593116 ## File path:

[GitHub] [spark] HeartSaVioR edited a comment on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite

2019-12-26 Thread GitBox
HeartSaVioR edited a comment on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite URL: https://github.com/apache/spark/pull/27001#issuecomment-569201728 > The fix itself LGTM, but do you know

[GitHub] [spark] cloud-fan commented on a change in pull request #27017: [SPARK-30359][CORE] Do not clear executorsPendingToRemove in CoarseGrainedSchedulerBackend.reset

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #27017: [SPARK-30359][CORE] Do not clear executorsPendingToRemove in CoarseGrainedSchedulerBackend.reset URL: https://github.com/apache/spark/pull/27017#discussion_r361592575 ## File path:

[GitHub] [spark] HeartSaVioR commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite

2019-12-26 Thread GitBox
HeartSaVioR commented on issue #27001: [SPARK-30345][SQL] Fix intermittent test failure (ConnectException) on ThriftServerQueryTestSuite/ThriftServerWithSparkContextSuite URL: https://github.com/apache/spark/pull/27001#issuecomment-569201728 > The fix itself LGTM, but do you know why

[GitHub] [spark] cloud-fan commented on a change in pull request #27017: [SPARK-30359][CORE] Do not clear executorsPendingToRemove in CoarseGrainedSchedulerBackend.reset

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #27017: [SPARK-30359][CORE] Do not clear executorsPendingToRemove in CoarseGrainedSchedulerBackend.reset URL: https://github.com/apache/spark/pull/27017#discussion_r361592575 ## File path:

[GitHub] [spark] SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
SparkQA commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569201596 **[Test build #115844 has

[GitHub] [spark] cloud-fan commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
cloud-fan commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569201345 Yea, I know we disable whole stage codegen when seeing `CodegenFallBack` now, I'm wondering if this is reasonable.

[GitHub] [spark] cloud-fan commented on a change in pull request #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#discussion_r361592123 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations

2019-12-26 Thread GitBox
cloud-fan commented on a change in pull request #26995: [SPARK-30341][SQL] Overflow check for interval arithmetic operations URL: https://github.com/apache/spark/pull/26995#discussion_r361591860 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569200556 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569200553 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569200556 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27008: [SPARK-30353][SQL] Add IsNotNull check in SimplifyBinaryComparison optimization URL: https://github.com/apache/spark/pull/27008#issuecomment-569200553 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569200374 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569200367 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map

2019-12-26 Thread GitBox
AmplabJenkins removed a comment on issue #27013: [SPARK-30356][SQL] Codegen support for the function str_to_map URL: https://github.com/apache/spark/pull/27013#issuecomment-569200367 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend

2019-12-26 Thread GitBox
AmplabJenkins commented on issue #26980: [SPARK-27348][CORE] HeartbeatReceiver should remove lost executors from CoarseGrainedSchedulerBackend URL: https://github.com/apache/spark/pull/26980#issuecomment-569200405 Merged build finished. Test PASSed.

  1   2   3   4   5   6   7   8   9   >