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.

Sign up for GitHub

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

Jump to bottom

Support distinct #430

Closed
BohuTANG opened this issue Apr 28, 2021 · 4 comments · Fixed by #752
Closed

Support distinct #430

BohuTANG opened this issue Apr 28, 2021 · 4 comments · Fixed by #752
Assignees
@sundy-li
Labels
A-query Area: databend query C-feature Category: feature prio: high High priority

Comments

@BohuTANG
Copy link
Member

BohuTANG commented Apr 28, 2021 •
edited
Loading

Summary

select count(distinct number) from (select number%3 as number from numbers_mt(100));
@BohuTANG BohuTANG added C-feature Category: feature prio: high High priority A-query Area: databend query SQL labels Apr 28, 2021
@sundy-li sundy-li self-assigned this Apr 28, 2021
@sundy-li sundy-li mentioned this issue Jun 7, 2021
Merged
@BohuTANG
Copy link
Member Author

BohuTANG commented Jun 7, 2021

@sundy-li

Do we also need to support other distinct?
Such sumDistinct, if need I am happy to open an issue

@sundy-li
Copy link
Member

sundy-li commented Jun 8, 2021

@sundy-li

Do we also need to support other distinct?
Such sumDistinct, if need I am happy to open an issue

Ok, though it's not MySQL style, but it's better than nothing. I'll try to implement a generic combinator for AggregateFunction, such as sumIf, sumDistinct

@BohuTANG
Copy link
Member Author

BohuTANG commented Jun 8, 2021

Sound interesting.
For functions, as long as they are helpful to the user, we should have them(MySQL's style function may not work well for analytics)!
BTW, I have add a functiuon track issue here #758

@BohuTANG
Copy link
Member Author

BohuTANG commented Jun 8, 2021

FYI, all the tracks also in Roadmap: #746

Maricaya pushed a commit to Maricaya/databend that referenced this issue Sep 29, 2024
@everpcpc
fix(cli): set default expand mode to auto (databendlabs#430)
a9ce7dc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sundy-li sundy-li

Labels
A-query Area: databend query C-feature Category: feature prio: high High priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[AggregateFunctions] Add uniq, count distinct aggregate functions sundy-li/databend
2 participants
@BohuTANG @sundy-li