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

More fixes #267

Open
wants to merge 5 commits into
base: gh-pages
from

Conversation

Projects
None yet
2 participants
@kyrretl
Contributor

kyrretl commented Oct 18, 2018

PR for the rest of the problems we ran into in our October workshop at the University of Oslo

Please see the commit messages for details

@@ -226,6 +226,16 @@ detail in [the next section]({{ site.github.url }}/06-agg/).
>
> to produce?
> What does it actually produce?
> > ## Solution
> >
> > The IN () function does not match NULL, and we only get results for the date '1927-02-08'.

This comment has been minimized.

@danmichaelo

danmichaelo Oct 19, 2018

Contributor

Perhaps it's even good to recap what IN really does at this point:

Suggested change Beta
> > The IN () function does not match NULL, and we only get results for the date '1927-02-08'.
> > We only get a result for the value '1927-02-08', not for NULL. This is because `dated IN ('1927-02-08', NULL)` is just a shorthand for `dated = '1927-02-08' OR dated = NULL`, and we now know that the latter condition (`dated = NULL`) is never true.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment