[GitHub] [spark] SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582255645 **[Test build #117894 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1178

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582255666 Merged build finished. Test FAILed. -

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582255666 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582255489 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582255494 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582255489 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582255494 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
SparkQA commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582255130 **[Test build #117896 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1178

[GitHub] [spark] ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter

2020-02-04 Thread GitBox
ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter URL: https://github.com/apache/spark/pull/27336#discussion_r375072678 ## File path: sql/core/src/main/scala/org/apache/spark/sql/functi

[GitHub] [spark] maropu commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write.

2020-02-04 Thread GitBox
maropu commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write. URL: https://github.com/apache/spark/pull/22163#issuecomment-582254464 Yea, thanks! This is an automated message fr

[GitHub] [spark] Ngone51 commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write.

2020-02-04 Thread GitBox
Ngone51 commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write. URL: https://github.com/apache/spark/pull/22163#issuecomment-582253898 Thanks for ping me. Let me try it in recent days. --

[GitHub] [spark] imback82 commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
imback82 commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375071874 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/cataly

[GitHub] [spark] beliefer commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
beliefer commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582253795 retest this please This is an automated

[GitHub] [spark] imback82 commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
imback82 commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375071833 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/an

[GitHub] [spark] cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375070537 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catal

[GitHub] [spark] cloud-fan commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter URL: https://github.com/apache/spark/pull/27336#discussion_r375070058 ## File path: sql/core/src/main/scala/org/apache/spark/sql/fun

[GitHub] [spark] SparkQA commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped

2020-02-04 Thread GitBox
SparkQA commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped URL: https://github.com/apache/spark/pull/27452#issuecomment-582251116 **[Test build #117895 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117895/testR

[GitHub] [spark] cloud-fan commented on a change in pull request #27151: [SPARK-26218][SQL][Follow up] Fix the corner case when casting float to Integer.

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27151: [SPARK-26218][SQL][Follow up] Fix the corner case when casting float to Integer. URL: https://github.com/apache/spark/pull/27151#discussion_r375068452 ## File path: sql/core/src/test/resources/sql-tests/inputs/postgreSQL/float4

[GitHub] [spark] AmplabJenkins removed a comment on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped URL: https://github.com/apache/spark/pull/27452#issuecomment-582249616 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] [spark] AmplabJenkins removed a comment on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped URL: https://github.com/apache/spark/pull/27452#issuecomment-582249608 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped URL: https://github.com/apache/spark/pull/27452#issuecomment-582249616 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] [spark] AmplabJenkins commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27452: [SPARK-30719][SQL] do not log warning if AQE is intentionally skipped URL: https://github.com/apache/spark/pull/27452#issuecomment-582249608 Merged build finished. Test PASSed. This is

[GitHub] [spark] ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter

2020-02-04 Thread GitBox
ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter URL: https://github.com/apache/spark/pull/27336#discussion_r375065772 ## File path: sql/core/src/main/scala/org/apache/spark/sql/functi

[GitHub] [spark] viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375066771 ## File path: sql/core/src/main/scala/org/apache/spark/sql/catalyst/anal

[GitHub] [spark] ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter

2020-02-04 Thread GitBox
ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter URL: https://github.com/apache/spark/pull/27336#discussion_r375065772 ## File path: sql/core/src/main/scala/org/apache/spark/sql/functi

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582247872 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582247874 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582247872 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582247874 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582247568 **[Test build #117894 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11789

[GitHub] [spark] ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter

2020-02-04 Thread GitBox
ueshin commented on a change in pull request #27336: [SPARK-28962][SQL][FOLLOW-UP] Add the parameter description for the Scala function API filter URL: https://github.com/apache/spark/pull/27336#discussion_r375065772 ## File path: sql/core/src/main/scala/org/apache/spark/sql/functi

[GitHub] [spark] maropu commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write.

2020-02-04 Thread GitBox
maropu commented on issue #22163: [SPARK-25166][CORE]Reduce the number of write operations for shuffle write. URL: https://github.com/apache/spark/pull/22163#issuecomment-582246736 kindly ping @Ngone51 This is an automated m

[GitHub] [spark] viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375065432 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst

[GitHub] [spark] viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
viirya commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375064422 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375064324 ## File path: core/src/test/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375058640 ## File path: core/src/main/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375061731 ## File path: core/src/test/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375056525 ## File path: core/src/main/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375064234 ## File path: core/src/test/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375057086 ## File path: core/src/main/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375061774 ## File path: core/src/test/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375058860 ## File path: core/src/main/scala/org/apache/spa

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375061419 ## File path: core/src/main/scala/org/apache/spa

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375058396 ## File path: core/src/main/scala/org/apache/sp

[GitHub] [spark] bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling.

2020-02-04 Thread GitBox
bmarcott commented on a change in pull request #27207: [WIP][SPARK-18886][CORE] Make Locality wait time measure resource under utilization due to delay scheduling. URL: https://github.com/apache/spark/pull/27207#discussion_r375057141 ## File path: core/src/main/scala/org/apache/spa

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582243845 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582243837 Merged build finished. Test FAILed. -

[GitHub] [spark] SparkQA removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
SparkQA removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235058 **[Test build #117891 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuild

[GitHub] [spark] SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582243798 **[Test build #117891 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1178

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582243845 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582243837 Merged build finished. Test FAILed.

[GitHub] [spark] maropu commented on a change in pull request #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED

2020-02-04 Thread GitBox
maropu commented on a change in pull request #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED URL: https://github.com/apache/spark/pull/27368#discussion_r375062180 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/a

[GitHub] [spark] maropu commented on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED

2020-02-04 Thread GitBox
maropu commented on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED URL: https://github.com/apache/spark/pull/27368#issuecomment-582243637 I left some minor comments and the other parts looks fine to me. ---

[GitHub] [spark] HeartSaVioR commented on issue #27333: [SPARK-29438][SS][FOLLOWUP] Add regression tests for Streaming Aggregation and flatMapGroupsWithState

2020-02-04 Thread GitBox
HeartSaVioR commented on issue #27333: [SPARK-29438][SS][FOLLOWUP] Add regression tests for Streaming Aggregation and flatMapGroupsWithState URL: https://github.com/apache/spark/pull/27333#issuecomment-582243278 cc. @tdas Kindly reminder. ---

[GitHub] [spark] maropu commented on a change in pull request #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED

2020-02-04 Thread GitBox
maropu commented on a change in pull request #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED URL: https://github.com/apache/spark/pull/27368#discussion_r375061952 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/a

[GitHub] [spark] maropu edited a comment on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED

2020-02-04 Thread GitBox
maropu edited a comment on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED URL: https://github.com/apache/spark/pull/27368#issuecomment-582242678 > I think we can keep empty Keys or Functions printed, which means the node has no Keys o

[GitHub] [spark] maropu commented on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED

2020-02-04 Thread GitBox
maropu commented on issue #27368: [SPARK-30651][SQL] Add detailed information for Aggregate operators in EXPLAIN FORMATTED URL: https://github.com/apache/spark/pull/27368#issuecomment-582242678 > I think we can keep empty Keys or Functions printed, which means the node has no Keys or Funct

[GitHub] [spark] AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582241707 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenki

[GitHub] [spark] cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375060713 ## File path: sql/core/src/test/resources/sql-tests/results/change-col

[GitHub] [spark] maropu commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
maropu commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#issuecomment-582241638 Ah, I see. Thanks. This is an aut

[GitHub] [spark] SparkQA removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
SparkQA removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238019 **[Test build #117893 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117893/testReport)** for PR 2746

[GitHub] [spark] AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582241705 Merged build finished. Test FAILed. This is an automated message from th

[GitHub] [spark] AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582241705 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] SparkQA commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
SparkQA commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582241648 **[Test build #117893 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117893/testReport)** for PR 27461 at com

[GitHub] [spark] AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582241707 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/

[GitHub] [spark] AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582241304 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582241295 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582241304 Test FAILed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582241295 Merged build finished. Test FAILed.

[GitHub] [spark] cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#discussion_r375059871 ## File path: sql/catalyst/src/main/antlr4/org/apache/spark/sql/catal

[GitHub] [spark] SparkQA removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
SparkQA removed a comment on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582184103 **[Test build #117867 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuil

[GitHub] [spark] SparkQA commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression

2020-02-04 Thread GitBox
SparkQA commented on issue #27429: [SPARK-28330][SQL] Support ANSI SQL: result offset clause in query expression URL: https://github.com/apache/spark/pull/27429#issuecomment-582240901 **[Test build #117867 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117

[GitHub] [spark] cloud-fan commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
cloud-fan commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#issuecomment-582240776 @maropu before 3.0 we only have hive style ALTER COLUMN syntax (which can change multiple prope

[GitHub] [spark] cloud-fan commented on a change in pull request #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
cloud-fan commented on a change in pull request #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#discussion_r375058966 ## File path: core/src/main/scala/org/apache/spark/internal/config/pac

[GitHub] [spark] maropu commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time

2020-02-04 Thread GitBox
maropu commented on issue #27444: [SPARK-30614][SQL] The native ALTER COLUMN syntax should change one property at a time URL: https://github.com/apache/spark/pull/27444#issuecomment-582238585 We don't need to keep the legacy(existing) behaviour? I think the current one still looks useful f

[GitHub] [spark] AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238325 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238330 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenk

[GitHub] [spark] AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238325 Merged build finished. Test PASSed. This is an automated message from th

[GitHub] [spark] AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238330 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job

[GitHub] [spark] SparkQA commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest

2020-02-04 Thread GitBox
SparkQA commented on issue #27461: [SPARK-30736][ML] One-Pass ChiSquareTest URL: https://github.com/apache/spark/pull/27461#issuecomment-582238019 **[Test build #117893 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/117893/testReport)** for PR 27461 at comm

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
HyukjinKwon commented on a change in pull request #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#discussion_r375054369 ## File path: R/create-rd.sh ## @@ -34,4 +34,5 @@ pushd "$FWDIR" > /

[GitHub] [spark] AmplabJenkins removed a comment on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables URL: https://github.com/apache/spark/pull/27391#issuecomment-582235385 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://ampl

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235376 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message URL: https://github.com/apache/spark/pull/27464#issuecomment-582235349 Test PASSed. Refer to this link for build results (access rights to CI server ne

[GitHub] [spark] AmplabJenkins removed a comment on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#issuecomment-582235352 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#issuecomment-582235357 Test PASSed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235372 Merged build finished. Test PASSed. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables URL: https://github.com/apache/spark/pull/27391#issuecomment-582235379 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message

2020-02-04 Thread GitBox
AmplabJenkins removed a comment on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message URL: https://github.com/apache/spark/pull/27464#issuecomment-582235344 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables URL: https://github.com/apache/spark/pull/27391#issuecomment-582235385 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.be

[GitHub] [spark] AmplabJenkins commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#issuecomment-582235357 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message URL: https://github.com/apache/spark/pull/27464#issuecomment-582235349 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#issuecomment-582235352 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235372 Merged build finished. Test PASSed.

[GitHub] [spark] HyukjinKwon commented on a change in pull request #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
HyukjinKwon commented on a change in pull request #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#discussion_r375054369 ## File path: R/create-rd.sh ## @@ -34,4 +34,5 @@ pushd "$FWDIR" > /

[GitHub] [spark] AmplabJenkins commented on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27464: [SPARK-29864][SQL][FOLLOWUP] Reference the config for the old behavior in error message URL: https://github.com/apache/spark/pull/27464#issuecomment-582235344 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27391: [SPARK-30612][SQL] Resolve qualified column name with v2 tables URL: https://github.com/apache/spark/pull/27391#issuecomment-582235379 Merged build finished. Test PASSed. This is an au

[GitHub] [spark] AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
AmplabJenkins commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235376 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade

2020-02-04 Thread GitBox
SparkQA commented on issue #27460: [SPARK-30733][R][HOTFIX] Fix SparkR tests per testthat and R version upgrade URL: https://github.com/apache/spark/pull/27460#issuecomment-582235058 **[Test build #117891 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/11789

[GitHub] [spark] SparkQA commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem`

2020-02-04 Thread GitBox
SparkQA commented on issue #27463: [SPARK-26700][Core][FOLLOWUP] Add config `spark.network.maxRemoteBlockSizeFetchToMem` URL: https://github.com/apache/spark/pull/27463#issuecomment-582235060 **[Test build #117890 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuil

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