Skip to content

Improve performance of aggregate expressions by implementing group accumulators #1627

Closed

Description

@andygrove

What is the problem the feature request solves?

DataFusion has a fast path for aggregates that implement group accumulators.

See #1275 (comment) for one example where we saw a 4x speedup.

We should apply this change to the following Comet aggregates.

  • stddev
  • correlation
  • covariance
  • variance

Describe the potential solution

No response

Additional context

No response

Activity

  1. added this to the 0.9.0 milestone on Apr 8, 2025
  2. dharanad commented on May 15, 2025

    @dharanad
    Contributor

    I would like implement group accumulators variance

  3. removed this from the 0.9.0 milestone on Jun 5, 2025
  4. self-assigned this
    on May 6, 2026
  5. added this to the 0.17.0 milestone on May 6, 2026
  6. moved this from Todo to In progress in Comet Developmenton May 6, 2026
  7. moved this from In progress to Done in Comet Developmenton May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions