Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Support multiple level aggregation in Calcite #3352

Closed
LantaoJin opened this issue Feb 26, 2025 · 1 comment
Closed

[FEATURE] Support multiple level aggregation in Calcite #3352

LantaoJin opened this issue Feb 26, 2025 · 1 comment
Labels
calcite calcite migration releated feature

Comments

@LantaoJin
Copy link
Member

LantaoJin commented Feb 26, 2025

What is the bug?

  1. source=EMP | stats avg(SAL) as avg_sal | stats avg(COMM) as avg_comm
  2. source = table | stats avg(response_time) as avg_response_time by host, service | stats avg(avg_response_time) as avg_host_response_time by service

Throws field [COMM] not found; input fields are: [avg_sal]

testMultipleLevelStats in CalcitePPLAggregationTest

@LantaoJin LantaoJin added bug Something isn't working untriaged calcite calcite migration releated and removed untriaged labels Feb 26, 2025
@LantaoJin LantaoJin changed the title [BUG] multiple level aggregate now work [FEATURE] Support multiple level aggregation Feb 26, 2025
@LantaoJin LantaoJin added feature and removed bug Something isn't working labels Feb 26, 2025
@LantaoJin LantaoJin changed the title [FEATURE] Support multiple level aggregation [FEATURE] Support multiple level aggregation in Calcite Feb 26, 2025
@LantaoJin
Copy link
Member Author

  1. should not be supported as an invalid query.
  2. have supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
calcite calcite migration releated feature
Projects
None yet
Development

No branches or pull requests

1 participant