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

fix count return null #6118

Merged
merged 9 commits into from
Jun 19, 2020
Merged

fix count return null #6118

merged 9 commits into from
Jun 19, 2020

Conversation

kimmking
Copy link
Member

Fixes #4680 .

Changes proposed in this pull request:

  • fix count return null

@kimmking kimmking added this to the 5.0.0 milestone Jun 18, 2020
@terrymanu terrymanu requested a review from tristaZero June 18, 2020 15:51
@tristaZero tristaZero merged commit d5a3224 into apache:master Jun 19, 2020
cmonkey pushed a commit to cmonkey/shardingsphere that referenced this pull request Jun 22, 2020
* fix count return null

* fix checkstyle

* fix unit test

* polish checkstyle

* polish code style

* fix in getMemoryResultSetRows

* fix code style

* extract method

* fix codestyle
@kimmking kimmking deleted the fix4680 branch July 2, 2020 12:47
sandynz pushed a commit to sandynz/shardingsphere that referenced this pull request Jul 10, 2020
* fix count return null

* fix checkstyle

* fix unit test

* polish checkstyle

* polish code style

* fix in getMemoryResultSetRows

* fix code style

* extract method

* fix codestyle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

count(distinct col) renturn null
2 participants