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

GROOVY-8901: Add DGSM countDistinct[By], sum[By], avg[By], etc. #831

Closed
wants to merge 1 commit into from
Closed

GROOVY-8901: Add DGSM countDistinct[By], sum[By], avg[By], etc. #831

wants to merge 1 commit into from

Conversation

daniellansun
Copy link
Contributor

No description provided.

@paulk-asert
Copy link
Contributor

paulk-asert commented Nov 24, 2018

Time to write some adoc doco for these new collector methods? ;-)

I'll try to have a bit of a better look on the plane flight to Java2days. I guess we should have avg for non-streams too. I also wonder whether there are any naming alterations needed to make stream and non-stream versions align? I'll go through some examples and let you know.

@daniellansun daniellansun changed the title GROOVY-8901: Add DGSM countDistinct[By], sum[By], avg[By] GROOVY-8901: Add DGSM countDistinct[By], sum[By], avg[By], etc. Nov 24, 2018
@daniellansun
Copy link
Contributor Author

I'll try to set aside some to write some doco.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants