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

Ep. 6: Explanation of count() could be improved. #305

Open
ajerrison-manchester opened this issue Nov 26, 2019 · 0 comments · May be fixed by #324
Open

Ep. 6: Explanation of count() could be improved. #305

ajerrison-manchester opened this issue Nov 26, 2019 · 0 comments · May be fixed by #324
Labels
type:clarification type:discussion type:enhancement

Comments

@ajerrison-manchester
Copy link
Contributor

@ajerrison-manchester ajerrison-manchester commented Nov 26, 2019

In episode 6 it is implied in the paragraph about count() that you would get the same result whatever field you counted, i.e. count(*) is the same as count(quant) is the same as count(person) etc. However, as the students learned in the previous episode aggregation functions do not include NULL values. I think it could be made a bit clearer in episode 6 that count(person) would not return 9 as there are NULL values in the person field. It would reinforce the learning of the previous episode and perhaps stop any confusion about what count does.

Please delete the text below before submitting your contribution.

@henrykironde henrykironde added type:clarification type:discussion type:enhancement labels Dec 3, 2019
@chrisroat chrisroat linked a pull request Sep 14, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:clarification type:discussion type:enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants