Skip to content

Test count(NULL) aggregation#10695

Closed
findepi wants to merge 1 commit intotrinodb:masterfrom
findepi:findepi/test-count-null-aggregation-863304
Closed

Test count(NULL) aggregation#10695
findepi wants to merge 1 commit intotrinodb:masterfrom
findepi:findepi/test-count-null-aggregation-863304

Conversation

@findepi
Copy link
Copy Markdown
Member

@findepi findepi commented Jan 19, 2022

This addresses a code TODO comment.

@findepi findepi requested review from kokosing and losipiuk January 19, 2022 15:13
@cla-bot cla-bot bot added the cla-signed label Jan 19, 2022
@findepi
Copy link
Copy Markdown
Member Author

findepi commented Jan 19, 2022

The test passes, because H2 has the same bug.

@findepi findepi closed this Jan 19, 2022
@findepi findepi reopened this Jan 19, 2022
@findepi
Copy link
Copy Markdown
Member Author

findepi commented Jan 19, 2022

The test passes, because H2 has the same bug.

im wrong

This addresses a code TODO comment.
@findepi findepi force-pushed the findepi/test-count-null-aggregation-863304 branch from 9bdc749 to b630c42 Compare January 19, 2022 15:27
@findepi
Copy link
Copy Markdown
Member Author

findepi commented Jan 19, 2022

Moved into #10663 .

@findepi findepi closed this Jan 19, 2022
@findepi findepi deleted the findepi/test-count-null-aggregation-863304 branch January 19, 2022 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant