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: execute "select count(*) from tbl" always getting zero #114

Merged
merged 2 commits into from
Aug 7, 2024

Conversation

harveyyue
Copy link
Contributor

Fix: #113

@WenyXu
Copy link
Collaborator

WenyXu commented Aug 5, 2024

Please run the make fmt

@WenyXu WenyXu requested a review from waynexia August 5, 2024 04:19
@harveyyue
Copy link
Contributor Author

Please run the make fmt

Done.

@WenyXu
Copy link
Collaborator

WenyXu commented Aug 6, 2024

@waynexia PTAL

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
@waynexia
Copy link
Collaborator

waynexia commented Aug 7, 2024

This seems to be an unexpected behavior of empty projection. This fix doesn't solves the root cause but it works anyway. I'm ok to merge it. Thanks @harveyyue

BTW, I just came across this だからもっと from @Jefffrey 😆
image

@waynexia waynexia merged commit 7844b9f into datafusion-contrib:main Aug 7, 2024
9 checks passed
waynexia added a commit that referenced this pull request Oct 24, 2024
* fix: execute "select count(*) from tbl" always getting zero

* docs: explain this special case in comment

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>

---------

Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Co-authored-by: Ruihang Xia <waynestxia@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Execute the statement select count(*) always getting 0
3 participants