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

Add erratum #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add erratum #2

wants to merge 2 commits into from

Conversation

shepard8
Copy link
Contributor

Maybe you would want to rework this example, since the IN operator does not really make sense (an equal would be enough) as there is only one value in the list.

Maybe you would want to rework this example, since the `IN` operator does not really make sense (an equal would be enough) as there is only one value in the list.
@@ -173,5 +173,6 @@ has been passionate about computer science since he was a 13-year-old, his first
#### Errata
* Page 82 (Dropping databases, first paragraph): **to drop a table** _should be_ **to drop a database**
* Page 581 (Section 5, heading): **The PostegreSQL System** _should be_ **The PostgreSQL System**
* Page 114 (Using the IN/NOT IN condition, last paragraph): **the values `pk=2` and `pk=4`** _should be_ **the single value `pk=11`**; **that have `pk=2` or `pk=4`** _should be_ **that have `category=11`**.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this could be even less clear, as it is written. Probably we should mention the whole statement, that is

query searches the records in the posts table that have pk=2 or pk=4.```
should be substitued with
```This statement extracts the value that corresponds to 'organe' from the category table and the external
query searches the records in the posts table that have such category, resulting in the tuples with pk=2 or pk=4

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

Successfully merging this pull request may close these issues.

2 participants