[GitHub] [spark] AmplabJenkins commented on issue #26869: [SPARK-30235][CORE] Switching off host local disk reading of shuffle blocks in case of useOldFetchProtocol

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26869: [SPARK-30235][CORE] Switching off host local disk reading of shuffle blocks in case of useOldFetchProtocol URL: https://github.com/apache/spark/pull/26869#issuecomment-566375359 Merged build finished. Test FAILed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26869: [SPARK-30235][CORE] Switching off host local disk reading of shuffle blocks in case of useOldFetchProtocol

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26869: [SPARK-30235][CORE] Switching off host local disk reading of shuffle blocks in case of useOldFetchProtocol URL: https://github.com/apache/spark/pull/26869#issuecomment-566375364 Test FAILed. Refer to this link for build results (access rig

[GitHub] [spark] cloud-fan commented on issue #26899: [SPARK-28332][SQL] Reserve init value -1 only when do min max statistics in SQLMetrics

2019-12-16 Thread GitBox
cloud-fan commented on issue #26899: [SPARK-28332][SQL] Reserve init value -1 only when do min max statistics in SQLMetrics URL: https://github.com/apache/spark/pull/26899#issuecomment-566377041 shall we update `SQLMetric.merge` to exclude -1 values? ---

[GitHub] [spark] fuwhu commented on a change in pull request #26684: [SPARK-30001][SQL] ResolveRelations should handle both V1 and V2 tables.

2019-12-16 Thread GitBox
fuwhu commented on a change in pull request #26684: [SPARK-30001][SQL] ResolveRelations should handle both V1 and V2 tables. URL: https://github.com/apache/spark/pull/26684#discussion_r358594236 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyz

[GitHub] [spark] SparkQA commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
SparkQA commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#issuecomment-566379151 **[Test build #115427 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuil

[GitHub] [spark] SparkQA commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
SparkQA commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566379164 **[Test build #115424 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115424/tes

[GitHub] [spark] AmplabJenkins commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566379363 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] [spark] sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#discussion_r358595078 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/H

[GitHub] [spark] AmplabJenkins commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566379359 Merged build finished. Test PASSed. This i

[GitHub] [spark] SparkQA removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566364589 **[Test build #115424 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115

[GitHub] [spark] AmplabJenkins removed a comment on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#issuecomment-566379409 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#issuecomment-566379417 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566379359 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26910: [SPARK-30154][ML] PySpark UDF to convert MLlib vectors to dense arrays URL: https://github.com/apache/spark/pull/26910#issuecomment-566379363 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins removed a comment on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#issuecomment-566379417 Test PASSed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#issuecomment-566379409 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
SparkQA commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566380557 **[Test build #115419 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115419/testReport)** for PR 26915 at c

[GitHub] [spark] SparkQA removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566350525 **[Test build #115419 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115419/testReport)** for PR 26

[GitHub] [spark] AmplabJenkins commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566380983 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//jo

[GitHub] [spark] AmplabJenkins commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566380977 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566380983 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jen

[GitHub] [spark] sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#discussion_r35859 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/H

[GitHub] [spark] AmplabJenkins removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26915: [INFRA] Reverts commit 56dcd79 and c216ef1 URL: https://github.com/apache/spark/pull/26915#issuecomment-566380977 Merged build finished. Test PASSed. This is an automated messa

[GitHub] [spark] sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function

2019-12-16 Thread GitBox
sandeep-katta commented on a change in pull request #25452: [SPARK-28710][SQL]to fix replace function, spark should call drop and create function URL: https://github.com/apache/spark/pull/25452#discussion_r358597386 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/H

[GitHub] [spark] SparkQA commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
SparkQA commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566382454 **[Test build #115421 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReque

[GitHub] [spark] iRakson edited a comment on issue #26863: [SPARK-30234]ADD FILE cannot add directories from sql CLI

2019-12-16 Thread GitBox
iRakson edited a comment on issue #26863: [SPARK-30234]ADD FILE cannot add directories from sql CLI URL: https://github.com/apache/spark/pull/26863#issuecomment-566144848 Test Suite added. cc @maropu @cloud-fan @HyukjinKwon @dongjoon-hyun --

[GitHub] [spark] SparkQA removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566352458 **[Test build #115421 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPu

[GitHub] [spark] AmplabJenkins commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566382939 Test PASSed. Refer to this link for build results (access rights to CI server needed)

[GitHub] [spark] AmplabJenkins commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566382935 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566382935 Merged build finished. Test PASSed. ---

[GitHub] [spark] SparkQA commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
SparkQA commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566383264 **[Test build #115423 has finished](https://amplab.cs.berkeley.edu/jenkins/job

[GitHub] [spark] AmplabJenkins removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566382939 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566383477 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566357719 **[Test build #115423 has started](https://amplab.cs.berkeley.edu/jenk

[GitHub] [spark] AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566383485 Test FAILed. Refer to this link for build results (access rights to CI

[GitHub] [spark] dongjoon-hyun closed pull request #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
dongjoon-hyun closed pull request #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916 This is an automated message from the Apach

[GitHub] [spark] AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566383477 Merged build finished. Test FAILed.

[GitHub] [spark] dongjoon-hyun commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite`

2019-12-16 Thread GitBox
dongjoon-hyun commented on issue #26916: [SPARK-25100][TEST][FOLLOWUP] Refactor test cases in `FileSuite` and `KryoSerializerSuite` URL: https://github.com/apache/spark/pull/26916#issuecomment-566383726 Thank you all. Merged to master. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566383485 Test FAILed. Refer to this link for build results (access righ

[GitHub] [spark] SparkQA commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
SparkQA commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566384392 **[Test build #115416 has finished](https://amplab.cs.berkeley.edu/jenkins/

[GitHub] [spark] SparkQA commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
SparkQA commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566384410 **[Test build #115415 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115415/testReport)** for PR 26809 a

[GitHub] [spark] SparkQA removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566322389 **[Test build #115415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115415/testReport)** for PR

[GitHub] [spark] SparkQA removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566335298 **[Test build #115416 has started](https://amplab.cs.berkeley.edu/j

[GitHub] [spark] AmplabJenkins commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566384772 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/

[GitHub] [spark] AmplabJenkins commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566384751 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566384765 Merged build finished. Test PASSed. This is an automated message fr

[GitHub] [spark] AmplabJenkins commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566384742 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566384765 Merged build finished. Test PASSed. This is an automated me

[GitHub] [spark] AmplabJenkins removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566384751 Test PASSed. Refer to this link for build results (access r

[GitHub] [spark] AmplabJenkins removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26809: [SPARK-30185][SQL] Implement Dataset.tail API URL: https://github.com/apache/spark/pull/26809#issuecomment-566384772 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/

[GitHub] [spark] AmplabJenkins removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26908: [SPARK-30269][SQL] Should use old partition stats to decide whether to update stats when analyzing partition URL: https://github.com/apache/spark/pull/26908#issuecomment-566384742 Merged build finished. Test PASSed. -

[GitHub] [spark] wangyum opened a new pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
wangyum opened a new pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919 ### What changes were proposed in this pull request? This PR update document for make Hive 2.3 dependency by default.

[GitHub] [spark] ulysses-you commented on issue #26831: [SPARK-30201][SQL] HiveOutputWriter standardOI should use ObjectInspectorCopyOption.DEFAULT

2019-12-16 Thread GitBox
ulysses-you commented on issue #26831: [SPARK-30201][SQL] HiveOutputWriter standardOI should use ObjectInspectorCopyOption.DEFAULT URL: https://github.com/apache/spark/pull/26831#issuecomment-566385401 Thanks! This is an auto

[GitHub] [spark] SparkQA commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
SparkQA commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386321 **[Test build #115428 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115428/test

[GitHub] [spark] AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386735 Merged build finished. Test PASSed. This i

[GitHub] [spark] AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386740 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampla

[GitHub] [spark] yujhe commented on issue #26807: [SPARK-30181][SQL] Only add string or integral type column to metastore partition filter

2019-12-16 Thread GitBox
yujhe commented on issue #26807: [SPARK-30181][SQL] Only add string or integral type column to metastore partition filter URL: https://github.com/apache/spark/pull/26807#issuecomment-566386817 @cloud-fan I have tested in `master` and `branch-2.4` branch with hive-1.2. `master` branch

[GitHub] [spark] AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386735 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386740 Test PASSed. Refer to this link for build results (access rights to CI server needed): https

[GitHub] [spark] yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#discussion_r358602970 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expression

[GitHub] [spark] yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#discussion_r358602970 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expression

[GitHub] [spark] AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566388743 Merged build finished. Test PASSed. This i

[GitHub] [spark] SparkQA removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566386321 **[Test build #115428 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115

[GitHub] [spark] AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566388749 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] [spark] SparkQA commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
SparkQA commented on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566388678 **[Test build #115428 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115428/tes

[GitHub] [spark] AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566388749 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#issuecomment-566388743 Merged build finished. Test PASSed. ---

[GitHub] [spark] cloud-fan commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
cloud-fan commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#discussion_r358606175 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressio

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
dongjoon-hyun commented on a change in pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#discussion_r358606356 ## File path: docs/building-spark.md ## @@ -83,13 +83,10 @@ Example:

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default

2019-12-16 Thread GitBox
dongjoon-hyun commented on a change in pull request #26919: [SPARK-30280][DOC] Update docs for make Hive 2.3 dependency by default URL: https://github.com/apache/spark/pull/26919#discussion_r358606489 ## File path: docs/building-spark.md ## @@ -83,13 +83,10 @@ Example:

[GitHub] [spark] yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
yaooqinn commented on a change in pull request #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#discussion_r358607182 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expression

[GitHub] [spark] cloud-fan commented on issue #26807: [SPARK-30181][SQL] Only add string or integral type column to metastore partition filter

2019-12-16 Thread GitBox
cloud-fan commented on issue #26807: [SPARK-30181][SQL] Only add string or integral type column to metastore partition filter URL: https://github.com/apache/spark/pull/26807#issuecomment-566391709 Did you use the latest branch-2.4? I'm a little confused about why the bug is still there. No

[GitHub] [spark] huaxingao commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
huaxingao commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566393965 retest this please -

[GitHub] [spark] planga82 commented on a change in pull request #26890: [SPARK-30039][SQL] CREATE FUNCTION should do multi-catalog resolution

2019-12-16 Thread GitBox
planga82 commented on a change in pull request #26890: [SPARK-30039][SQL] CREATE FUNCTION should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26890#discussion_r358613325 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveSes

[GitHub] [spark] planga82 commented on a change in pull request #26890: [SPARK-30039][SQL] CREATE FUNCTION should do multi-catalog resolution

2019-12-16 Thread GitBox
planga82 commented on a change in pull request #26890: [SPARK-30039][SQL] CREATE FUNCTION should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26890#discussion_r358613325 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveSes

[GitHub] [spark] AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566394205 Merged build finished. Test PASSed.

[GitHub] [spark] maropu commented on issue #26917: [SPARK-30278][SQL][DOC] Update Spark SQL document menu for new changes

2019-12-16 Thread GitBox
maropu commented on issue #26917: [SPARK-30278][SQL][DOC] Update Spark SQL document menu for new changes URL: https://github.com/apache/spark/pull/26917#issuecomment-566394194 How about adding the docs about the SQL case, too? https://github.com/apache/spark/pull/25464 ---

[GitHub] [spark] AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566394210 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566394205 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566394210 Test PASSed. Refer to this link for build results (access righ

[GitHub] [spark] SparkQA commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams

2019-12-16 Thread GitBox
SparkQA commented on issue #26838: [SPARK-30144][ML][PySpark] Make MultilayerPerceptronClassificationModel extend MultilayerPerceptronParams URL: https://github.com/apache/spark/pull/26838#issuecomment-566395699 **[Test build #115429 has started](https://amplab.cs.berkeley.edu/jenkins/job/

[GitHub] [spark] AmplabJenkins commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566398230 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566398221 Merged build finished. Test PASSed. ---

[GitHub] [spark] bmarcott commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
bmarcott commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566398156 back from vacation and updated the PR trying to address all the comments

[GitHub] [spark] AmplabJenkins removed a comment on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566398230 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType

2019-12-16 Thread GitBox
amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType URL: https://github.com/apache/spark/pull/26811#discussion_r358617660 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFun

[GitHub] [spark] AmplabJenkins removed a comment on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566398221 Merged build finished. Test PASSed. ---

[GitHub] [spark] amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType

2019-12-16 Thread GitBox
amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType URL: https://github.com/apache/spark/pull/26811#discussion_r358076319 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFun

[GitHub] [spark] bmarcott commented on a change in pull request #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
bmarcott commented on a change in pull request #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#discussion_r358617814 ## File path: core/src/main/scala/org/apache/

[GitHub] [spark] amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType

2019-12-16 Thread GitBox
amanomer commented on a change in pull request #26811: [SPARK-29600][SQL] ArrayContains function may return incorrect result for DecimalType URL: https://github.com/apache/spark/pull/26811#discussion_r358618026 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFun

[GitHub] [spark] SparkQA commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized

2019-12-16 Thread GitBox
SparkQA commented on issue #26696: [WIP][SPARK-18886][CORE] Make locality wait time be the time since a TSM's available slots were fully utilized URL: https://github.com/apache/spark/pull/26696#issuecomment-566399899 **[Test build #115430 has started](https://amplab.cs.berkeley.edu/jenkins

[GitHub] [spark] zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors

2019-12-16 Thread GitBox
zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors URL: https://github.com/apache/spark/pull/26858#discussion_r358620083 ## File path: mllib/src/main/scala/org/apache/spark/ml/feature/

[GitHub] [spark] zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors

2019-12-16 Thread GitBox
zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors URL: https://github.com/apache/spark/pull/26858#discussion_r358620329 ## File path: mllib/src/main/scala/org/apache/spark/ml/feature/

[GitHub] [spark] SparkQA commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
SparkQA commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566401668 **[Test build #115420 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115420/t

[GitHub] [spark] SparkQA removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566350550 **[Test build #115420 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1

[GitHub] [spark] AmplabJenkins commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566402102 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566402107 Test PASSed. Refer to this link for build results (access rights to CI server needed): http

[GitHub] [spark] AmplabJenkins commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566402107 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors

2019-12-16 Thread GitBox
zhengruifeng commented on a change in pull request #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors URL: https://github.com/apache/spark/pull/26858#discussion_r358620968 ## File path: mllib/src/main/scala/org/apache/spark/ml/feature/

[GitHub] [spark] AmplabJenkins removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566402102 Merged build finished. Test PASSed. -

<    3   4   5   6   7   8   9   >