-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
fix issue #4816 some aggregate function work incorrectly in cluster mode #4817
Conversation
mengjinglei
commented
Nov 17, 2015
- CHANGELOG.md updated
- Rebased/mergable
- Tests pass
- Sign CLA (if not already signed)
Is there any way to unit test this? |
I noticed that there is no unit test case for |
|
+1 @dgnorton -- can you please also review? |
CLA confirmed signed. |
+1 |
fbf3bea
to
51a17b1
Compare
51a17b1
to
9b6cdb4
Compare
Thanks @mengjinglei |
fix issue #4816 some aggregate function work incorrectly in cluster mode