[GitHub] [spark] AmplabJenkins commented on issue #27267: [WIP][SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27267: [WIP][SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575725054 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575725026 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575725019 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575710491 Can one of the admins verify this patch?

[GitHub] [spark] SparkQA commented on issue #27267: [WIP][SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
SparkQA commented on issue #27267: [WIP][SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575724568 **[Test build #116960 has

[GitHub] [spark] dongjoon-hyun commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575724445 Thank you for pinging me, @ajithme . This

[GitHub] [spark] SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575724538 **[Test build #116961 has

[GitHub] [spark] dongjoon-hyun commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575724477 ok to test This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #27267: [SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27267: [SPARK-30556][SQL] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575716494 Can one of the admins verify this patch?

[GitHub] [spark] dongjoon-hyun commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575723223 ok to test This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575722277 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575722288 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575722288 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575722277 Merged build finished. Test PASSed. This is an

[GitHub] [spark] dongjoon-hyun commented on issue #27262: [SPARK-30551][SQL] Disable comparison for interval type

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27262: [SPARK-30551][SQL] Disable comparison for interval type URL: https://github.com/apache/spark/pull/27262#issuecomment-575720954 Also, cc @maropu This is an automated message from the

[GitHub] [spark] AmplabJenkins removed a comment on issue #27268: [SPARK-30553] fix structured-streaming java example error

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27268: [SPARK-30553] fix structured-streaming java example error URL: https://github.com/apache/spark/pull/27268#issuecomment-575718875 Can one of the admins verify this patch? This

[GitHub] [spark] AmplabJenkins commented on issue #27268: [SPARK-30553] fix structured-streaming java example error

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27268: [SPARK-30553] fix structured-streaming java example error URL: https://github.com/apache/spark/pull/27268#issuecomment-575719255 Can one of the admins verify this patch? This is an

[GitHub] [spark] AmplabJenkins commented on issue #27268: [SPARK-30553] fix structured-streaming java example error

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27268: [SPARK-30553] fix structured-streaming java example error URL: https://github.com/apache/spark/pull/27268#issuecomment-575718875 Can one of the admins verify this patch? This is an

[GitHub] [spark] huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#discussion_r368047041 ## File path: docs/sql-ref-syntax-ddl-create-table-datasource.md ## @@ -0,0 +1,113

[GitHub] [spark] huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#discussion_r368047112 ## File path: docs/sql-ref-syntax-ddl-create-table-hiveformat.md ## @@ -0,0 +1,121

[GitHub] [spark] AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575716681 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575716688 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575716688 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] bettermouse opened a new pull request #27268: [SPARK-30553] fix structured-streaming java example error

2020-01-17 Thread GitBox
bettermouse opened a new pull request #27268: [SPARK-30553] fix structured-streaming java example error URL: https://github.com/apache/spark/pull/27268 # What changes were proposed in this pull request? #30553 fix structured-streaming java example error. It does not clean up old

[GitHub] [spark] AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575716681 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575716494 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575716018 Can one of the admins verify this patch?

[GitHub] [spark] SparkQA commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
SparkQA commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575716162 **[Test build #116959 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/116959/testReport)** for PR

[GitHub] [spark] AmplabJenkins commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575716018 Can one of the admins verify this patch?

[GitHub] [spark] ajithme commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
ajithme commented on issue #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267#issuecomment-575715631 cc @dongjoon-hyun @srowen @HyukjinKwon @gatorsmile @cloud-fan

[GitHub] [spark] ajithme opened a new pull request #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext

2020-01-17 Thread GitBox
ajithme opened a new pull request #27267: [SPARK-30556] Copy sparkContext.localproperties to child thread inSubqueryExec.executionContext URL: https://github.com/apache/spark/pull/27267 ### What changes were proposed in this pull request? In

[GitHub] [spark] AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575713801 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575713791 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575713791 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575713801 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

2020-01-17 Thread GitBox
SparkQA commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575713239 **[Test build #116958 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/116958/testReport)** for PR

[GitHub] [spark] huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#discussion_r368041028 ## File path: docs/sql-ref-syntax-ddl-create-table.md ## @@ -19,4 +19,14 @@

[GitHub] [spark] MaxGekk commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
MaxGekk commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575711644 @srowen Main goal is code polish. Potentially, it could improve performance slightly because `Seq` builder is

[GitHub] [spark] huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#discussion_r368040930 ## File path: docs/sql-ref-syntax-ddl-create-table-hiveformat.md ## @@ -0,0 +1,121

[GitHub] [spark] huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
huaxingao commented on a change in pull request #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#discussion_r368040877 ## File path: docs/sql-ref-syntax-ddl-create-table-datasource.md ## @@ -0,0 +1,113

[GitHub] [spark] AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575710491 Can one of the admins verify this patch?

[GitHub] [spark] cloud-fan closed pull request #26675: [SPARK-30041][SQL][WEBUI] Add Codegen Stage Id to Stage DAG visualization in Web UI

2020-01-17 Thread GitBox
cloud-fan closed pull request #26675: [SPARK-30041][SQL][WEBUI] Add Codegen Stage Id to Stage DAG visualization in Web UI URL: https://github.com/apache/spark/pull/26675 This is an automated message from the Apache Git

[GitHub] [spark] cloud-fan commented on issue #26675: [SPARK-30041][SQL][WEBUI] Add Codegen Stage Id to Stage DAG visualization in Web UI

2020-01-17 Thread GitBox
cloud-fan commented on issue #26675: [SPARK-30041][SQL][WEBUI] Add Codegen Stage Id to Stage DAG visualization in Web UI URL: https://github.com/apache/spark/pull/26675#issuecomment-575710489 thanks, merging to master! This

[GitHub] [spark] AmplabJenkins removed a comment on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575709887 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575709887 Can one of the admins verify this patch?

[GitHub] [spark] MaxGekk commented on a change in pull request #27239: [SPARK-30530][SQL] Fix filter pushdown for bad CSV records

2020-01-17 Thread GitBox
MaxGekk commented on a change in pull request #27239: [SPARK-30530][SQL] Fix filter pushdown for bad CSV records URL: https://github.com/apache/spark/pull/27239#discussion_r368038138 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/UnivocityParser.scala

[GitHub] [spark] ajithme commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
ajithme commented on issue #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266#issuecomment-575708817 cc @dongjoon-hyun @HyukjinKwon @srowen @gatorsmile

[GitHub] [spark] JoshRosen commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor

2020-01-17 Thread GitBox
JoshRosen commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor URL: https://github.com/apache/spark/pull/27088#issuecomment-575708699 Let me take a look at this today / check my notes (busy week for me, hence my slow response).

[GitHub] [spark] cloud-fan commented on a change in pull request #26805: [SPARK-15616][SQL] Add optimizer rule PruneHiveTablePartitions

2020-01-17 Thread GitBox
cloud-fan commented on a change in pull request #26805: [SPARK-15616][SQL] Add optimizer rule PruneHiveTablePartitions URL: https://github.com/apache/spark/pull/26805#discussion_r368037093 ## File path:

[GitHub] [spark] ajithme opened a new pull request #27266: [SPARK-22590] Broadcast thread propagates the local properties to task

2020-01-17 Thread GitBox
ajithme opened a new pull request #27266: [SPARK-22590] Broadcast thread propagates the local properties to task URL: https://github.com/apache/spark/pull/27266 ### What changes were proposed in this pull request? In

[GitHub] [spark] srowen commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
srowen commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575708379 Seems OK; is it just some code polish or does it make an impact on performance?

[GitHub] [spark] AmplabJenkins removed a comment on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575703968 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575703984 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575703968 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575703984 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575703399 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575703399 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575703417 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575703417 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
SparkQA commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575703415 **[Test build #116957 has

[GitHub] [spark] SparkQA removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
SparkQA removed a comment on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575606489 **[Test build #116942 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/116942/testReport)**

[GitHub] [spark] cloud-fan commented on a change in pull request #27263: [SPARK-30433][SQL][FOLLOW-UP] Optimize collect conflict plans

2020-01-17 Thread GitBox
cloud-fan commented on a change in pull request #27263: [SPARK-30433][SQL][FOLLOW-UP] Optimize collect conflict plans URL: https://github.com/apache/spark/pull/27263#discussion_r368031213 ## File path:

[GitHub] [spark] SparkQA commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
SparkQA commented on issue #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#issuecomment-575702615 **[Test build #116942 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/116942/testReport)** for PR

[GitHub] [spark] cloud-fan commented on a change in pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
cloud-fan commented on a change in pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#discussion_r368029518 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
cloud-fan commented on a change in pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#discussion_r368029518 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575700744 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575700959 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575700736 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575700959 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575700965 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575700965 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
cloud-fan commented on issue #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265#issuecomment-575700671 cc @xianyinxin @tdas This

[GitHub] [spark] AmplabJenkins commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575700736 Merged build finished. Test PASSed. This

[GitHub] [spark] cloud-fan opened a new pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table

2020-01-17 Thread GitBox
cloud-fan opened a new pull request #27265: [SPARK-30555][SQL] MERGE INTO insert action should only access columns from source table URL: https://github.com/apache/spark/pull/27265 ### What changes were proposed in this pull request? when resolving the `Assignment` of insert

[GitHub] [spark] AmplabJenkins commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575700744 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
SparkQA removed a comment on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575608835 **[Test build #116943 has

[GitHub] [spark] SparkQA commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
SparkQA commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575700295 **[Test build #116956 has

[GitHub] [spark] SparkQA commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source

2020-01-17 Thread GitBox
SparkQA commented on issue #27039: [SPARK-29174][SQL] Support LOCAL in INSERT OVERWRITE DIRECTORY to data source URL: https://github.com/apache/spark/pull/27039#issuecomment-575700188 **[Test build #116943 has

[GitHub] [spark] SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575698294 **[Test build #116955 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575698425 Merged build finished. Test PASSed. This

[GitHub] [spark] SparkQA removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
SparkQA removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575693808 **[Test build #116955 has

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575698438 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575698438 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575698425 Merged build finished. Test PASSed. This is an

[GitHub] [spark] MaxGekk commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
MaxGekk commented on issue #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264#issuecomment-575697550 @HyukjinKwon @srowen Please, review this PR.

[GitHub] [spark] MaxGekk opened a new pull request #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser`

2020-01-17 Thread GitBox
MaxGekk opened a new pull request #27264: [SPARK-30554][SQL] Return `Iterable` from `FailureSafeParser.rawParser` URL: https://github.com/apache/spark/pull/27264 ### What changes were proposed in this pull request? Changed signature of `rawParser` passed to `FailureSafeParser`. I

[GitHub] [spark] dongjoon-hyun commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor URL: https://github.com/apache/spark/pull/27088#issuecomment-575696033 Hi, @JoshRosen . Could you address @maropu 's comment?

[GitHub] [spark] viirya commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together

2020-01-17 Thread GitBox
viirya commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together URL: https://github.com/apache/spark/pull/27256#issuecomment-575695907 This error looks a bit different to previous ones. Looks like a connection error? Is it repeatedly occurred?

[GitHub] [spark] dongjoon-hyun edited a comment on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor

2020-01-17 Thread GitBox
dongjoon-hyun edited a comment on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor URL: https://github.com/apache/spark/pull/27088#issuecomment-575694847 Thank you for pinging me, @maropu .

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575694382 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins removed a comment on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575694376 Merged build finished. Test PASSed. This

[GitHub] [spark] dongjoon-hyun commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27088: [SPARK-30413][SQL] Avoid WrappedArray roundtrip in GenericArrayData constructor URL: https://github.com/apache/spark/pull/27088#issuecomment-575694847 Thank you for pinging me, @maropu . I'll review today.

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575694376 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
AmplabJenkins commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575694382 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command

2020-01-17 Thread GitBox
SparkQA commented on issue #26759: [SPARK-28794][SQL][DOC] Documentation for Create table Command URL: https://github.com/apache/spark/pull/26759#issuecomment-575693808 **[Test build #116955 has

[GitHub] [spark] dongjoon-hyun commented on issue #27251: [SPARK-30539][PYTHON][SQL] Add DataFrame.tail in PySpark

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27251: [SPARK-30539][PYTHON][SQL] Add DataFrame.tail in PySpark URL: https://github.com/apache/spark/pull/27251#issuecomment-575690581 Also, cc @BryanCutler . This is an automated message

[GitHub] [spark] dongjoon-hyun closed pull request #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together

2020-01-17 Thread GitBox
dongjoon-hyun closed pull request #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together URL: https://github.com/apache/spark/pull/27256 This is an automated message from the Apache Git Service. To respond to

[GitHub] [spark] dongjoon-hyun commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together URL: https://github.com/apache/spark/pull/27256#issuecomment-575685986 All the other tests (except the flaky CRAN incoming check) passed. I'll merge this to branch-2.4.

[GitHub] [spark] dongjoon-hyun commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together

2020-01-17 Thread GitBox
dongjoon-hyun commented on issue #27256: [SPARK-30333][BUILD][FOLLOWUP][2.4] Update sbt build together URL: https://github.com/apache/spark/pull/27256#issuecomment-575685444 Hi, @viirya . CRAN incoming feasibility test seems to fail again. > * checking CRAN incoming feasibility

[GitHub] [spark] srowen commented on a change in pull request #27261: [SPARK-30503][ML] OnlineLDAOptimizer does not handle persistance correctly

2020-01-17 Thread GitBox
srowen commented on a change in pull request #27261: [SPARK-30503][ML] OnlineLDAOptimizer does not handle persistance correctly URL: https://github.com/apache/spark/pull/27261#discussion_r368006636 ## File path:

<    1   2   3   4   5   6   7   8   9   10   >