[GitHub] [spark] AmplabJenkins commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566426374 Merged build finished. Test PASSed. Th

[GitHub] [spark] AmplabJenkins removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566426374 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566426376 Test PASSed. Refer to this link for build results (access rights to CI server needed): ht

[GitHub] [spark] AmplabJenkins commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566426376 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://am

[GitHub] [spark] SparkQA commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
SparkQA commented on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566425781 **[Test build #115425 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115425

[GitHub] [spark] SparkQA removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID

2019-12-16 Thread GitBox
SparkQA removed a comment on issue #26918: [SPARK-30279][SQL] Support 32 or more grouping attributes for GROUPING_ID URL: https://github.com/apache/spark/pull/26918#issuecomment-566366273 **[Test build #115425 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder

[GitHub] [spark] steven-aerts commented on a change in pull request #26907: [SPARK-30267][SQL] Avro arrays can be of any List

2019-12-16 Thread GitBox
steven-aerts commented on a change in pull request #26907: [SPARK-30267][SQL] Avro arrays can be of any List URL: https://github.com/apache/spark/pull/26907#discussion_r358641861 ## File path: external/avro/src/main/scala/org/apache/spark/sql/avro/AvroDeserializer.scala ##

[GitHub] [spark] zhengruifeng commented on issue #26803: [SPARK-30178][ML] RobustScaler support large numFeatures

2019-12-16 Thread GitBox
zhengruifeng commented on issue #26803: [SPARK-30178][ML] RobustScaler support large numFeatures URL: https://github.com/apache/spark/pull/26803#issuecomment-566419509 Moreover, for existing impl, we can use `reduceByKey`/`aggregateByKey` with feature index as a key to generate a `RDD[Quan

[GitHub] [spark] AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566419176 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566419171 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566419171 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566419176 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] SparkQA commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
SparkQA commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566418536 **[Test build #115437 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1154

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

2019-12-16 Thread GitBox
cloud-fan 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_r358633880 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFu

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

2019-12-16 Thread GitBox
cloud-fan 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_r358633880 ## File path: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFu

[GitHub] [spark] zhengruifeng commented on issue #26803: [SPARK-30178][ML] RobustScaler support large numFeatures

2019-12-16 Thread GitBox
zhengruifeng commented on issue #26803: [SPARK-30178][ML] RobustScaler support large numFeatures URL: https://github.com/apache/spark/pull/26803#issuecomment-566414715 test code: ```scala import org.apache.spark.ml.linalg._ import org.apache.spark.ml.feature._ import org.apache.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566413592 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566413585 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566413585 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566413592 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] SparkQA commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
SparkQA commented on issue #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921#issuecomment-566412969 **[Test build #115436 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1154

[GitHub] [spark] imback82 opened a new pull request #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first

2019-12-16 Thread GitBox
imback82 opened a new pull request #26921: [SPARK-30282][SQL] UnresolvedV2Relation should be resolved to temp view first URL: https://github.com/apache/spark/pull/26921 ### What changes were proposed in this pull request? This is a part of effort to make the relation lookup b

[GitHub] [spark] AmplabJenkins removed a comment on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#issuecomment-566411082 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#issuecomment-566411091 Test PASSed. Refer to this link for build results (access rights to CI server needed): h

[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-566410838 Test PASSed. Refer to this link for build results (access rights to CI server need

[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-566410827 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#issuecomment-566411082 Merged build finished. Test PASSed. Th

[GitHub] [spark] AmplabJenkins commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#issuecomment-566411091 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://a

[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-566410827 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-566410838 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile

2019-12-16 Thread GitBox
SparkQA commented on issue #26905: [SPARK-30266][SQL] Avoid overflow and match error in ApproximatePercentile URL: https://github.com/apache/spark/pull/26905#issuecomment-566410649 **[Test build #115435 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115435/

[GitHub] [spark] SparkQA 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
SparkQA 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-566379151 **[Test build #115427 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReq

[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-566410409 **[Test build #115427 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBui

[GitHub] [spark] wypoon commented on a change in pull request #26895: [SPARK-17398][SQL] Fix ClassCastException when querying partitioned JSON table

2019-12-16 Thread GitBox
wypoon commented on a change in pull request #26895: [SPARK-17398][SQL] Fix ClassCastException when querying partitioned JSON table URL: https://github.com/apache/spark/pull/26895#discussion_r358628516 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/TableReader.scala

[GitHub] [spark] yaooqinn commented on issue #26412: [SPARK-29774][SQL] Date and Timestamp type +/- null should be null as Postgres

2019-12-16 Thread GitBox
yaooqinn commented on issue #26412: [SPARK-29774][SQL] Date and Timestamp type +/- null should be null as Postgres URL: https://github.com/apache/spark/pull/26412#issuecomment-566410341 Hi @gengliangwang, thanks for your suggestion, I have updated the description. Can you check whether it

[GitHub] [spark] AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566408882 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566408889 Test PASSed. Refer to this link for build results (access right

[GitHub] [spark] AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566408889 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566408882 Merged build finished. Test PASSed. -

[GitHub] [spark] SparkQA 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
SparkQA 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-566408400 **[Test build #115434 has started](https://amplab.cs.berkeley.edu/jenkins/job

[GitHub] [spark] SparkQA commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
SparkQA commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566408383 **[Test build #115433 has started](https://amplab.cs.berkeley.edu/jenkins/job/S

[GitHub] [spark] JkSelf commented on a change in pull request #26434: [SPARK-29544] [SQL] optimize skewed partition based on data size

2019-12-16 Thread GitBox
JkSelf commented on a change in pull request #26434: [SPARK-29544] [SQL] optimize skewed partition based on data size URL: https://github.com/apache/spark/pull/26434#discussion_r358626209 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ReduceNumShufflePartition

[GitHub] [spark] JkSelf commented on a change in pull request #26434: [SPARK-29544] [SQL] optimize skewed partition based on data size

2019-12-16 Thread GitBox
JkSelf commented on a change in pull request #26434: [SPARK-29544] [SQL] optimize skewed partition based on data size URL: https://github.com/apache/spark/pull/26434#discussion_r358626105 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ReduceNumShufflePartition

[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-566406639 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566406677 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566406685 Test PASSed. Refer to this link for build results (access right

[GitHub] [spark] AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566406677 Merged build finished. Test PASSed. -

[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-566406649 Test PASSed. Refer to this link for build results (access rig

[GitHub] [spark] AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566406685 Test PASSed. Refer to this link for build results (access rights to CI

[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-566406649 Test PASSed. Refer to this link for build results (access rights to C

[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-566406639 Merged build finished. Test PASSed. ---

[GitHub] [spark] SparkQA commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
SparkQA commented on issue #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920#issuecomment-566406291 **[Test build #115432 has started](https://amplab.cs.berkeley.edu/jenkins/job/S

[GitHub] [spark] HeartSaVioR commented on a change in pull request #22952: [SPARK-20568][SS] Provide option to clean up completed files in streaming query

2019-12-16 Thread GitBox
HeartSaVioR commented on a change in pull request #22952: [SPARK-20568][SS] Provide option to clean up completed files in streaming query URL: https://github.com/apache/spark/pull/22952#discussion_r358624511 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/strea

[GitHub] [spark] HeartSaVioR opened a new pull request #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource

2019-12-16 Thread GitBox
HeartSaVioR opened a new pull request #26920: [SPARK-30281][SS] Consider partitioned/recursive option while verifying archive path on FileStreamSource URL: https://github.com/apache/spark/pull/26920 ### What changes were proposed in this pull request? This patch renews the verificati

[GitHub] [spark] AmplabJenkins removed a comment on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404516 Merged build finished. Test PASSed. ---

[GitHub] [spark] AmplabJenkins removed a comment on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
AmplabJenkins removed a comment on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404525 Test PASSed. Refer to this link for build results (access rights to CI server needed): h

[GitHub] [spark] AmplabJenkins commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404525 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://a

[GitHub] [spark] AmplabJenkins commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
AmplabJenkins commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404516 Merged build finished. Test PASSed. Th

[GitHub] [spark] beliefer commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
beliefer commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404539 @cloud-fan @maropu @viirya @dongjoon-hyun In order to make review more convenient and easy, the current PR o

[GitHub] [spark] zhengruifeng commented on issue #26882: [SPARK-30247][PySpark] GaussianMixtureModel in py side should expose gaussian

2019-12-16 Thread GitBox
zhengruifeng commented on issue #26882: [SPARK-30247][PySpark] GaussianMixtureModel in py side should expose gaussian URL: https://github.com/apache/spark/pull/26882#issuecomment-566404111 Sorry for the late reply. It seems we need to add a corresponding class `MultivariateGaussian` con

[GitHub] [spark] SparkQA commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression

2019-12-16 Thread GitBox
SparkQA commented on issue #26656: [SPARK-27986][SQL] Support ANSI SQL filter clause for aggregate expression URL: https://github.com/apache/spark/pull/26656#issuecomment-566404107 **[Test build #115431 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/115431/

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

2019-12-16 Thread GitBox
yaooqinn commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566403842 thanks for merging and reviewing. This is an

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

2019-12-16 Thread GitBox
maropu commented on issue #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906#issuecomment-566403582 Thanks! Merged to master. This is an automated

[GitHub] [spark] maropu closed pull request #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache

2019-12-16 Thread GitBox
maropu closed pull request #26906: [SPARK-30066][SQL][FOLLOWUP] Remove size field for interval column cache URL: https://github.com/apache/spark/pull/26906 This is an automated message from the Apache Git Service. To respond

[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. -

[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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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 #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] 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] 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] 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] 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] 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] 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] 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] 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

  1   2   3   4   5   6   7   8   9   >