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
Open

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

ajerrison-manchester opened this issue Nov 26, 2019 · 0 comments
Labels
type:clarification Suggest change for make lesson clearer type:discussion Discussion or feedback about the lesson type:enhancement Propose enhancement to the lesson

Comments

@ajerrison-manchester
Copy link
Contributor

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 Suggest change for make lesson clearer type:discussion Discussion or feedback about the lesson type:enhancement Propose enhancement to the lesson labels Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:clarification Suggest change for make lesson clearer type:discussion Discussion or feedback about the lesson type:enhancement Propose enhancement to the lesson
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants