[GitHub] [spark] AmplabJenkins removed a comment on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog URL: https://github.com/apache/spark/pull/25104#issuecomment-524879672 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog URL: https://github.com/apache/spark/pull/25104#issuecomment-524879672 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog URL: https://github.com/apache/spark/pull/25104#issuecomment-524879679 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog URL: https://github.com/apache/spark/pull/25104#issuecomment-524824809 **[Test build #109730 has

[GitHub] [spark] SparkQA commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog

2019-08-26 Thread GitBox
SparkQA commented on issue #25104: [SPARK-28341][SQL] create a public API for V2SessionCatalog URL: https://github.com/apache/spark/pull/25104#issuecomment-524879351 **[Test build #109730 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109730/testReport)**

[GitHub] [spark] cloud-fan commented on issue #25347: [SPARK-28610][SQL] Allow having a decimal buffer for long sum

2019-08-26 Thread GitBox
cloud-fan commented on issue #25347: [SPARK-28610][SQL] Allow having a decimal buffer for long sum URL: https://github.com/apache/spark/pull/25347#issuecomment-524876439 @mgaido91 can you do a simple microbenchmark? If the performance overhead is not significant, we can use decimal as sum

[GitHub] [spark] srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#discussion_r317621710 ## File path:

[GitHub] [spark] srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#discussion_r317621165 ## File path:

[GitHub] [spark] srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
srowen commented on a change in pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#discussion_r317621622 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#discussion_r317619714 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#discussion_r317619345 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#issuecomment-524872836 Can one of the admins verify this patch?

[GitHub] [spark] SparkQA commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
SparkQA commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#issuecomment-524873878 **[Test build #109736 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#issuecomment-524872616 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#issuecomment-524872836 Can one of the admins verify this patch? This is

[GitHub] [spark] AmplabJenkins commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583#issuecomment-524872616 Can one of the admins verify this patch? This is

[GitHub] [spark] HeartSaVioR opened a new pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate

2019-08-26 Thread GitBox
HeartSaVioR opened a new pull request #25583: [MINOR][SS] Reuse KafkaSourceInitialOffsetWriter to deduplicate URL: https://github.com/apache/spark/pull/25583 ### What changes were proposed in this pull request? This patch proposes to reuse KafkaSourceInitialOffsetWriter to remove

[GitHub] [spark] attilapiros commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host

2019-08-26 Thread GitBox
attilapiros commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host URL: https://github.com/apache/spark/pull/25299#discussion_r317614716 ## File path:

[GitHub] [spark] attilapiros commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host

2019-08-26 Thread GitBox
attilapiros commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host URL: https://github.com/apache/spark/pull/25299#discussion_r317614594 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524868589 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] SparkQA removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524861704 **[Test build #109735 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524868574 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524868574 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524868589 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] SparkQA commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
SparkQA commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524868447 **[Test build #109735 has

[GitHub] [spark] HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer

2019-08-26 Thread GitBox
HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer URL: https://github.com/apache/spark/pull/22138#discussion_r317612424 ## File path:

[GitHub] [spark] viirya edited a comment on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns

2019-08-26 Thread GitBox
viirya edited a comment on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns URL: https://github.com/apache/spark/pull/25570#issuecomment-524844659 hmm I am not sure if ALTER VIEW AS should take the schema of the new query, or it should keep

[GitHub] [spark] HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer

2019-08-26 Thread GitBox
HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer URL: https://github.com/apache/spark/pull/22138#discussion_r317610953 ## File path:

[GitHub] [spark] viirya edited a comment on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns

2019-08-26 Thread GitBox
viirya edited a comment on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns URL: https://github.com/apache/spark/pull/25570#issuecomment-524844659 hmm I am not sure if ALTER VIEW AS should take the schema of the new query, or it should keep

[GitHub] [spark] SparkQA removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524855919 **[Test build #109734 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524865107 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task retry tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524865113 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524865107 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524865113 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524865022 **[Test build #109734 has

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317608095 ## File path:

[GitHub] [spark] HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer

2019-08-26 Thread GitBox
HeartSaVioR commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer URL: https://github.com/apache/spark/pull/22138#discussion_r317608101 ## File path:

[GitHub] [spark] HyukjinKwon closed pull request #25567: [SPARK-28527][SQL][TEST] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
HyukjinKwon closed pull request #25567: [SPARK-28527][SQL][TEST] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567 This is an automated message from the Apache Git

[GitHub] [spark] gaborgsomogyi commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer

2019-08-26 Thread GitBox
gaborgsomogyi commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer URL: https://github.com/apache/spark/pull/22138#discussion_r317607287 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317607046 ## File path:

[GitHub] [spark] HyukjinKwon commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
HyukjinKwon commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524864119 Merged to master.

[GitHub] [spark] SparkQA commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
SparkQA commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524861704 **[Test build #109735 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524860908 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524860918 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524860908 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#issuecomment-524860918 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317603133 ## File path:

[GitHub] [spark] abellina commented on a change in pull request #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling

2019-08-26 Thread GitBox
abellina commented on a change in pull request #25403: [SPARK-28679][YARN] changes to setResourceInformation to handle empty resources and reflection error handling URL: https://github.com/apache/spark/pull/25403#discussion_r317602682 ## File path:

[GitHub] [spark] gaborgsomogyi commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
gaborgsomogyi commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524856702 cc @HeartSaVioR to reflect discussion

[GitHub] [spark] gaborgsomogyi commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer

2019-08-26 Thread GitBox
gaborgsomogyi commented on a change in pull request #22138: [SPARK-25151][SS] Apply Apache Commons Pool to KafkaDataConsumer URL: https://github.com/apache/spark/pull/22138#discussion_r317597605 ## File path:

[GitHub] [spark] SparkQA commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA commented on issue #25582: [WIP][SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524855919 **[Test build #109734 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524855486 Test PASSed. Refer to this link for build results

[GitHub] [spark] AmplabJenkins removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524855478 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524855478 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524855486 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array

2019-08-26 Thread GitBox
maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array URL: https://github.com/apache/spark/pull/25172#discussion_r317593758 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array

2019-08-26 Thread GitBox
maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array URL: https://github.com/apache/spark/pull/25172#discussion_r317596308 ## File path:

[GitHub] [spark] SparkQA removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524746857 **[Test build #109722 has

[GitHub] [spark] SparkQA commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server

2019-08-26 Thread GitBox
SparkQA commented on issue #25567: [SPARK-28527][SQL][TEST][test-maven][test-hadoop3.2] Re-run all the tests in SQLQueryTestSuite via Thrift Server URL: https://github.com/apache/spark/pull/25567#issuecomment-524854721 **[Test build #109722 has

[GitHub] [spark] xueyumusic commented on issue #21985: [SPARK-24884][SQL] add regexp_extract_all support

2019-08-26 Thread GitBox
xueyumusic commented on issue #21985: [SPARK-24884][SQL] add regexp_extract_all support URL: https://github.com/apache/spark/pull/21985#issuecomment-524854113 Hi, @maropu , I updated PR description based on new template, thanks.

[GitHub] [spark] maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array

2019-08-26 Thread GitBox
maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array URL: https://github.com/apache/spark/pull/25172#discussion_r317594610 ## File path: sql/core/src/test/scala/org/apache/spark/sql/StringFunctionsSuite.scala

[GitHub] [spark] maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array

2019-08-26 Thread GitBox
maropu commented on a change in pull request #25172: [SPARK-28412][SQL] ANSI SQL: OVERLAY function support byte array URL: https://github.com/apache/spark/pull/25172#discussion_r317593758 ## File path:

[GitHub] [spark] juliuszsompolski commented on a change in pull request #25277: [SPARK-28637][SQL] Thriftserver support interval type

2019-08-26 Thread GitBox
juliuszsompolski commented on a change in pull request #25277: [SPARK-28637][SQL] Thriftserver support interval type URL: https://github.com/apache/spark/pull/25277#discussion_r317591260 ## File path:

[GitHub] [spark] viirya commented on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns

2019-08-26 Thread GitBox
viirya commented on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns URL: https://github.com/apache/spark/pull/25570#issuecomment-524845828 Btw, in the original JIRA comment, someone claims that Hive supports duplicate column names. I want

[GitHub] [spark] viirya commented on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns

2019-08-26 Thread GitBox
viirya commented on issue #25570: [SPARK-23519][SQL] create view should work from query with duplicate output columns URL: https://github.com/apache/spark/pull/25570#issuecomment-524844659 hmm I am not sure if ALTER VIEW AS should take the schema of the new query, or it should keep

[GitHub] [spark] cloud-fan closed pull request #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
cloud-fan closed pull request #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [spark] cloud-fan commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
cloud-fan commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524843332 thanks, merging to master! This is an automated message from the Apache

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524840380 Test FAILed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524840372 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524835684 **[Test build #109732 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524837276 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524840372 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524840341 **[Test build #109732 has

[GitHub] [spark] SparkQA commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
SparkQA commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524840399 **[Test build #109733 has

[GitHub] [spark] AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524840380 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524839790 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524839790 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524839778 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524839778 Merged build finished. Test PASSed. This is an

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317572985 ## File path:

[GitHub] [spark] maropu commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
maropu commented on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524838511 retest this please This is an automated message from the

[GitHub] [spark] maropu removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions

2019-08-26 Thread GitBox
maropu removed a comment on issue #20965: [SPARK-21870][SQL] Split aggregation code into small functions URL: https://github.com/apache/spark/pull/20965#issuecomment-524743934 retet this please This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524837102 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524837276 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524837102 Can one of the admins verify this patch?

[GitHub] [spark] SparkQA commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
SparkQA commented on issue #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582#issuecomment-524835684 **[Test build #109732 has

[GitHub] [spark] gaborgsomogyi opened a new pull request #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used

2019-08-26 Thread GitBox
gaborgsomogyi opened a new pull request #25582: [SPARK-28875][DSTREAMS][SS][TESTS] Add Task rety tests to make sure new consumer used URL: https://github.com/apache/spark/pull/25582 ### What changes were proposed in this pull request? When Task retry happens with Kafka source then it's

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317572985 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog

2019-08-26 Thread GitBox
cloud-fan commented on a change in pull request #25507: [SPARK-28667][SQL] Support InsertInto through the V2SessionCatalog URL: https://github.com/apache/spark/pull/25507#discussion_r317519172 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524829832 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524829825 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524829825 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524829832 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on issue #25470: [SPARK-28751][Core][WIP] Improve java serializer deserialization performance

2019-08-26 Thread GitBox
cloud-fan commented on issue #25470: [SPARK-28751][Core][WIP] Improve java serializer deserialization performance URL: https://github.com/apache/spark/pull/25470#issuecomment-524829460 @ConeyLiu do you mean we have to use a fresh `ObjectInputStream` for each deserialization because it's

[GitHub] [spark] SparkQA removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
SparkQA removed a comment on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524761956 **[Test build #109725 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109725/testReport)** for PR

[GitHub] [spark] SparkQA commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command

2019-08-26 Thread GitBox
SparkQA commented on issue #24759: [SPARK-27395][SQL] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-524829275 **[Test build #109725 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/109725/testReport)** for PR 24759 at

[GitHub] [spark] AmplabJenkins removed a comment on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#issuecomment-524828411 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
AmplabJenkins removed a comment on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#issuecomment-524828403 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#issuecomment-524828411 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion

2019-08-26 Thread GitBox
AmplabJenkins commented on issue #25581: [SPARK-28495][SQL] Introduce ANSI store assignment policy for table insertion URL: https://github.com/apache/spark/pull/25581#issuecomment-524828403 Merged build finished. Test PASSed.

<    3   4   5   6   7   8   9   10   11   >