[GitHub] [spark] kiszk commented on a change in pull request #24795: [SPARK-27945][SQL] Minimal changes to support columnar processing

2019-06-15 Thread GitBox
kiszk commented on a change in pull request #24795: [SPARK-27945][SQL] Minimal changes to support columnar processing URL: https://github.com/apache/spark/pull/24795#discussion_r294037693 ## File path: sql/core/src/test/scala/org/apache/spark/sql/SparkSessionExtensionSuite.scala

[GitHub] [spark] SparkQA commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
SparkQA commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357324 **[Test build #106541 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/106541/testReport)** for PR

[GitHub] [spark] HyukjinKwon commented on issue #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2

2019-06-15 Thread GitBox
HyukjinKwon commented on issue #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2 URL: https://github.com/apache/spark/pull/24327#issuecomment-502360092 @dongjoon-hyun, @rdblue, @cloud-fan, let me know if there are any major comments to address that I missed. If that's not

[GitHub] [spark] HyukjinKwon commented on issue #24826: [SPARK-27870][SQL][PYTHON] Add a runtime buffer size configuration for Pandas UDFs

2019-06-15 Thread GitBox
HyukjinKwon commented on issue #24826: [SPARK-27870][SQL][PYTHON] Add a runtime buffer size configuration for Pandas UDFs URL: https://github.com/apache/spark/pull/24826#issuecomment-502360206 Merged to master. Thanks all for reviewing this.

[GitHub] [spark] AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502370195 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502370192 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502370192 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502370195 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
SparkQA removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357324 **[Test build #106541 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/106541/testReport)**

[GitHub] [spark] SparkQA commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
SparkQA commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502370054 **[Test build #106541 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/106541/testReport)** for PR

[GitHub] [spark] mgaido91 commented on issue #24873: [SPARK-28052][SQL] Make `ArrayExists` follow the three-valued boolean logic.

2019-06-15 Thread GitBox
mgaido91 commented on issue #24873: [SPARK-28052][SQL] Make `ArrayExists` follow the three-valued boolean logic. URL: https://github.com/apache/spark/pull/24873#issuecomment-502343944 LGTM as well, thanks @ueshin ! This is

[GitHub] [spark] Ngone51 commented on issue #24848: [SPARK-28014][core] All waiting apps will be changed to the wrong state of Running after master changed.

2019-06-15 Thread GitBox
Ngone51 commented on issue #24848: [SPARK-28014][core] All waiting apps will be changed to the wrong state of Running after master changed. URL: https://github.com/apache/spark/pull/24848#issuecomment-502343941 @jerryshao Can you have a look please ? This seems a change introduced by you.

[GitHub] [spark] HyukjinKwon commented on issue #24867: [SPARK-28041][PYTHON] Increase minimum supported Pandas to 0.23.2

2019-06-15 Thread GitBox
HyukjinKwon commented on issue #24867: [SPARK-28041][PYTHON] Increase minimum supported Pandas to 0.23.2 URL: https://github.com/apache/spark/pull/24867#issuecomment-502348918 Looks good to me too but can we hold it for few days just to let more people read the discussion (since now it's

[GitHub] [spark] hvanhovell commented on issue #24706: [SPARK-23128][SQL] A new approach to do adaptive execution in Spark SQL

2019-06-15 Thread GitBox
hvanhovell commented on issue #24706: [SPARK-23128][SQL] A new approach to do adaptive execution in Spark SQL URL: https://github.com/apache/spark/pull/24706#issuecomment-502350698 Merging to master. @maryannxue @carsonwang thanks for all the hard work!

[GitHub] [spark] chenjunjiedada opened a new pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
chenjunjiedada opened a new pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879 ## What changes were proposed in this pull request? This pr is used to support using hostpath/PV volume mounts as

[GitHub] [spark] HeartSaVioR commented on issue #23850: [SPARK-26949][SS] Prevent 'purge' to remove needed batch files in CompactibleFileStreamLog

2019-06-15 Thread GitBox
HeartSaVioR commented on issue #23850: [SPARK-26949][SS] Prevent 'purge' to remove needed batch files in CompactibleFileStreamLog URL: https://github.com/apache/spark/pull/23850#issuecomment-502369627 Thanks all for reviewing and merging!

[GitHub] [spark] hvanhovell closed pull request #24706: [SPARK-23128][SQL] A new approach to do adaptive execution in Spark SQL

2019-06-15 Thread GitBox
hvanhovell closed pull request #24706: [SPARK-23128][SQL] A new approach to do adaptive execution in Spark SQL URL: https://github.com/apache/spark/pull/24706 This is an automated message from the Apache Git Service. To

[GitHub] [spark] HyukjinKwon closed pull request #24826: [SPARK-27870][SQL][PYTHON] Add a runtime buffer size configuration for Pandas UDFs

2019-06-15 Thread GitBox
HyukjinKwon closed pull request #24826: [SPARK-27870][SQL][PYTHON] Add a runtime buffer size configuration for Pandas UDFs URL: https://github.com/apache/spark/pull/24826 This is an automated message from the Apache Git

[GitHub] [spark] rxin commented on issue #24232: [SPARK-27297] [SQL] Add higher order functions to scala API

2019-06-15 Thread GitBox
rxin commented on issue #24232: [SPARK-27297] [SQL] Add higher order functions to scala API URL: https://github.com/apache/spark/pull/24232#issuecomment-502367342 I feel it's ok to have these functions. Fills a gap. This

[GitHub] [spark] gengliangwang commented on issue #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2

2019-06-15 Thread GitBox
gengliangwang commented on issue #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2 URL: https://github.com/apache/spark/pull/24327#issuecomment-502367477 @dongjoon-hyun @rdblue @cloud-fan @mallman @HyukjinKwon @gatorsmile @jaceklaskowski Thanks for the review!

[GitHub] [spark] AmplabJenkins commented on issue #24878: [SPARK-27042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24878: [SPARK-27042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24878#issuecomment-502347864 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24878: [SPARK-27042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24878: [SPARK-27042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24878#issuecomment-502347730 Can one of the admins verify this patch?

[GitHub] [spark] peter-toth commented on issue #24842: [SPARK-28002][SQL] Support WITH clause column aliases

2019-06-15 Thread GitBox
peter-toth commented on issue #24842: [SPARK-28002][SQL] Support WITH clause column aliases URL: https://github.com/apache/spark/pull/24842#issuecomment-502357103 Thanks @dongjoon-hyun @liancheng @gatorsmile @hvanhovell for the review. I have prepared

[GitHub] [spark] AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357209 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357207 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357209 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24831: [SPARK-19799][SQL] Better support for WITH clause URL: https://github.com/apache/spark/pull/24831#issuecomment-502357207 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] HyukjinKwon closed pull request #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2

2019-06-15 Thread GitBox
HyukjinKwon closed pull request #24327: [SPARK-27418][SQL] Migrate Parquet to File Data Source V2 URL: https://github.com/apache/spark/pull/24327 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] srowen commented on a change in pull request #24843: [SPARK-28004][UI] Update jquery to 3.4.1

2019-06-15 Thread GitBox
srowen commented on a change in pull request #24843: [SPARK-28004][UI] Update jquery to 3.4.1 URL: https://github.com/apache/spark/pull/24843#discussion_r294049535 ## File path: core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ## @@ -366,9

[GitHub] [spark] mengxr commented on issue #24643: [SPARK-26412][PySpark][SQL] Allow Pandas UDF to take an iterator of pd.Series or an iterator of tuple of pd.Series

2019-06-15 Thread GitBox
mengxr commented on issue #24643: [SPARK-26412][PySpark][SQL] Allow Pandas UDF to take an iterator of pd.Series or an iterator of tuple of pd.Series URL: https://github.com/apache/spark/pull/24643#issuecomment-502375891 Merged into master. Thanks!

[GitHub] [spark] chenjunjiedada closed pull request #24878: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
chenjunjiedada closed pull request #24878: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24878 This is an automated message from the Apache Git Service. To

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502366407 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502366437 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502366047 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502366407 Can one of the admins verify this patch?

[GitHub] [spark] srowen closed pull request #24857: [MINOR][CORE] Remove unused variables, unused imports, etc.

2019-06-15 Thread GitBox
srowen closed pull request #24857: [MINOR][CORE] Remove unused variables, unused imports, etc. URL: https://github.com/apache/spark/pull/24857 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] dongjoon-hyun commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
dongjoon-hyun commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502385254 ok to test This is an automated message

[GitHub] [spark] dongjoon-hyun commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
dongjoon-hyun commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502385247 Thank you for making a PR, @chenjunjiedada !

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386224 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386223 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
SparkQA removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502385349 **[Test build #106542 has

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#discussion_r294054867 ## File path:

[GitHub] [spark] felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#discussion_r294055265 ## File path:

[GitHub] [spark] SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502385349 **[Test build #106542 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502366437 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386223 Merged build finished. Test FAILed. This

[GitHub] [spark] SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386218 **[Test build #106542 has

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386224 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502387495 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502387497 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502387487 Kubernetes integration test status success URL:

[GitHub] [spark] MaxGekk commented on issue #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types

2019-06-15 Thread GitBox
MaxGekk commented on issue #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types URL: https://github.com/apache/spark/pull/24872#issuecomment-502394796 Just in case, how about to introduce new parameter for `toInt`, `toLong` and etc. (to existing or new

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types URL: https://github.com/apache/spark/pull/24872#discussion_r294054152 ## File path: docs/sql-migration-guide-upgrade.md ## @@

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types URL: https://github.com/apache/spark/pull/24872#discussion_r294054076 ## File path:

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24872: [SPARK-28023][SQL] Trim the string when cast string type to Boolean/Numeric types URL: https://github.com/apache/spark/pull/24872#discussion_r294054186 ## File path:

[GitHub] [spark] dongjoon-hyun closed pull request #24873: [SPARK-28052][SQL] Make `ArrayExists` follow the three-valued boolean logic.

2019-06-15 Thread GitBox
dongjoon-hyun closed pull request #24873: [SPARK-28052][SQL] Make `ArrayExists` follow the three-valued boolean logic. URL: https://github.com/apache/spark/pull/24873 This is an automated message from the Apache Git

[GitHub] [spark] SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
SparkQA commented on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502386558 Kubernetes integration test starting URL:

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502387497 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#issuecomment-502387495 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880#issuecomment-502395191 Can one of the admins verify this patch?

[GitHub] [spark] Andrew-Crosby opened a new pull request #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
Andrew-Crosby opened a new pull request #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880 ## What changes were proposed in this pull request? Modifies the HuberAggregator class so that a copy of

[GitHub] [spark] dongjoon-hyun commented on issue #24842: [SPARK-28002][SQL] Support WITH clause column aliases

2019-06-15 Thread GitBox
dongjoon-hyun commented on issue #24842: [SPARK-28002][SQL] Support WITH clause column aliases URL: https://github.com/apache/spark/pull/24842#issuecomment-502384413 Thanks. No problem, @peter-toth ! This is an automated

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24853: [SPARK-28029][SQL][TEST] Port int2.sql

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24853: [SPARK-28029][SQL][TEST] Port int2.sql URL: https://github.com/apache/spark/pull/24853#discussion_r294054768 ## File path: sql/core/src/test/resources/sql-tests/inputs/pgSQL/int2.sql ## @@ -0,0 +1,129 @@ +-- +--

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24853: [SPARK-28029][SQL][TEST] Port int2.sql

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24853: [SPARK-28029][SQL][TEST] Port int2.sql URL: https://github.com/apache/spark/pull/24853#discussion_r294054768 ## File path: sql/core/src/test/resources/sql-tests/inputs/pgSQL/int2.sql ## @@ -0,0 +1,129 @@ +-- +--

[GitHub] [spark] felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#discussion_r294055334 ## File path:

[GitHub] [spark] felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage

2019-06-15 Thread GitBox
felixcheung commented on a change in pull request #24879: [SPARK-28042][K8S] Support using hostpath volume mount as local storage URL: https://github.com/apache/spark/pull/24879#discussion_r294055385 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880#issuecomment-502395502 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880#issuecomment-502395191 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880#issuecomment-502395502 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24880: [SPARK-28062][ML] Avoid unnecessary copy of coefficients vector in HuberAggregator URL: https://github.com/apache/spark/pull/24880#issuecomment-502395527 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins removed a comment on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql URL: https://github.com/apache/spark/pull/24881#issuecomment-502409386 Can one of the admins verify this patch? This is an automated

[GitHub] [spark] SparkQA commented on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql

2019-06-15 Thread GitBox
SparkQA commented on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql URL: https://github.com/apache/spark/pull/24881#issuecomment-502417036 **[Test build #106545 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/106545/testReport)** for PR 24881 at

[GitHub] [spark] dongjoon-hyun commented on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql

2019-06-15 Thread GitBox
dongjoon-hyun commented on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql URL: https://github.com/apache/spark/pull/24881#issuecomment-502417006 ok to test This is an automated message from the Apache Git Service. To

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #24846: [SPARK-28012][SQL] Hive UDF supports literal struct type

2019-06-15 Thread GitBox
dongjoon-hyun commented on a change in pull request #24846: [SPARK-28012][SQL] Hive UDF supports literal struct type URL: https://github.com/apache/spark/pull/24846#discussion_r294066883 ## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala

[GitHub] [spark] SparkQA commented on issue #24851: [SPARK-27303][GRAPH] Add PropertyGraph construction API

2019-06-15 Thread GitBox
SparkQA commented on issue #24851: [SPARK-27303][GRAPH] Add PropertyGraph construction API URL: https://github.com/apache/spark/pull/24851#issuecomment-502418028 **[Test build #106547 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/106547/testReport)** for

[GitHub] [spark] SparkQA commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
SparkQA commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418034 **[Test build #106548 has

[GitHub] [spark] SparkQA commented on issue #24440: [SPARK-27545] [SQL] Uncache table needs to delete the temporary view …

2019-06-15 Thread GitBox
SparkQA commented on issue #24440: [SPARK-27545] [SQL] Uncache table needs to delete the temporary view … URL: https://github.com/apache/spark/pull/24440#issuecomment-502418035 **[Test build #106549 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #24637: [SPARK-27707][SQL] Prune unnecessary nested fields from Generate to address performance issue in explode

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24637: [SPARK-27707][SQL] Prune unnecessary nested fields from Generate to address performance issue in explode URL: https://github.com/apache/spark/pull/24637#issuecomment-502417950 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418813 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
SparkQA removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418034 **[Test build #106548 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418811 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418811 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418813 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] SparkQA commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive

2019-06-15 Thread GitBox
SparkQA commented on issue #24566: [SPARK-27667][SQL] Get the current database from spark catalog instead of querying the Hive URL: https://github.com/apache/spark/pull/24566#issuecomment-502418792 **[Test build #106548 has

[GitHub] [spark] SparkQA removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
SparkQA removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502418666 **[Test build #106550 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502419134 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502419135 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502419134 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502419135 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts

2019-06-15 Thread GitBox
SparkQA commented on issue #24601: [SPARK-27702][K8S] Allow using some alternatives for service accounts URL: https://github.com/apache/spark/pull/24601#issuecomment-502419124 **[Test build #106550 has

[GitHub] [spark] dongjoon-hyun opened a new pull request #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
dongjoon-hyun opened a new pull request #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882 ## What changes were proposed in this pull request? This PR aims to replace deprecated `.newInstance()` in DSv2

[GitHub] [spark] SparkQA commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
SparkQA commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502421776 **[Test build #106551 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502421660 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502421659 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502422269 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502422269 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
SparkQA removed a comment on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502421776 **[Test build #106551 has

[GitHub] [spark] SparkQA commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
SparkQA commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502422267 **[Test build #106551 has

[GitHub] [spark] AmplabJenkins commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs`

2019-06-15 Thread GitBox
AmplabJenkins commented on issue #24882: [SPARK-28063][SQL] Replace deprecated `.newInstance()` in DSv2 `Catalogs` URL: https://github.com/apache/spark/pull/24882#issuecomment-502422270 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql

2019-06-15 Thread GitBox
AmplabJenkins removed a comment on issue #24881: [SPARK-23160][SQL][TEST] Port window.sql URL: https://github.com/apache/spark/pull/24881#issuecomment-502422923 Merged build finished. Test FAILed. This is an automated

  1   2   >