Skip to content

[SPARK-27974][SQL] Support ANSI Aggregate Function: array_agg#34866

Closed
beliefer wants to merge 6 commits intoapache:masterfrom
beliefer:SPARK-27974
Closed

[SPARK-27974][SQL] Support ANSI Aggregate Function: array_agg#34866
beliefer wants to merge 6 commits intoapache:masterfrom
beliefer:SPARK-27974

Conversation

@beliefer
Copy link
Contributor

@beliefer beliefer commented Dec 11, 2021

@github-actions github-actions bot added the SQL label Dec 11, 2021
@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50560/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50560/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50561/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Test build #146085 has finished for PR 34866 at commit 584418c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50561/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Test build #146086 has finished for PR 34866 at commit 5a87b37.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50564/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50564/

@SparkQA
Copy link

SparkQA commented Dec 11, 2021

Test build #146089 has finished for PR 34866 at commit 7dd4c44.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@beliefer
Copy link
Contributor Author

ping @MaxGekk @gengliangwang cc @cloud-fan

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would write only one test with GROUP BY, which has duplicated values

(1,4),(2,3),(1,4),(2,4)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@SparkQA
Copy link

SparkQA commented Dec 16, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50762/

@SparkQA
Copy link

SparkQA commented Dec 16, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50762/

@SparkQA
Copy link

SparkQA commented Dec 16, 2021

Test build #146290 has finished for PR 34866 at commit a895ad7.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50777/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50777/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Test build #146305 has finished for PR 34866 at commit 590279c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50787/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50787/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Test build #146314 has finished for PR 34866 at commit 990472a.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50797/

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50797/

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in 026158a Dec 17, 2021
@beliefer
Copy link
Contributor Author

@cloud-fan Thank you very much.

@SparkQA
Copy link

SparkQA commented Dec 17, 2021

Test build #146323 has finished for PR 34866 at commit bbba0d0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

Does this PR introduce any user-facing change?
'No'. New feature.

Please be more careful when writing PR descriptions. How can this be not user-facing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments