Skip to content

[BUG] Replace duplicated aggregation logic with aggregateWithTrimming() #4925

@ishaoxy

Description

@ishaoxy

What is the bug?
In the current implementation of buildStreamWindowJoinPlan in the Streamstats command, the aggregation for window functions was partially re-implemented inside this method. But we should just use aggregateWithTrimming() to avoid duplicating existing functionality.

How can one reproduce the bug?
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

What is the expected behavior?
A clear and concise description of what you expected to happen.

What is your host/environment?

  • OS: [e.g. iOS]
  • Version [e.g. 22]
  • Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.
``

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions