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

planner: mpp-agg output wrong result for empty set in the mock handler of unistore #50893

Closed
AilinKid opened this issue Feb 1, 2024 · 0 comments · Fixed by #50961 · May be fixed by #50856 or #50982
Closed

planner: mpp-agg output wrong result for empty set in the mock handler of unistore #50893

AilinKid opened this issue Feb 1, 2024 · 0 comments · Fixed by #50961 · May be fixed by #50856 or #50982
Assignees
Labels
severity/moderate sig/planner SIG: Planner type/bug The issue is confirmed as a bug.

Comments

@AilinKid
Copy link
Contributor

AilinKid commented Feb 1, 2024

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

say we got a count[MPP] for mpp handler in unistore, and we split it as two partial aggregation.

sum(#1) as #2 [MPP]
   +---count(source) as #1  [MPP]
             +--- source (let it be empty set)  [MPP]

currently mpp handler will output nil (chunk size as 0 for #1) facing another empty set of source
currently mpp handler will output nil (chunk size as 0 for #2) facing another empty set

2. What did you expect to see? (Required)

we get 0 of #1 even for an empty set of source
we get 0 of #2 because it has 1 row as 0.

3. What did you see instead (Required)

return empty set

4. What is your TiDB version? (Required)

master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment