Skip to content

queries added - #2

Open
LevyMo wants to merge 2 commits into
mainfrom
lesson2
Open

queries added#2
LevyMo wants to merge 2 commits into
mainfrom
lesson2

Conversation

@LevyMo

@LevyMo LevyMo commented Jul 9, 2024

Copy link
Copy Markdown
Owner

No description provided.

@dsinn dsinn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Almost there! I'd just like to see some minor fixes to answers 6 and 8.

Comment thread sql2.txt Outdated
Comment on lines +68 to +70
VALUES (10248, 23, 1),
(10248, 24, 1),
(10248, 25, 1);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

10248 looks to be the ID of the oldest order in the starter data, rather than the new order from answer 5. Could you fix this? (In real life, modifying the wrong order could lead to an angry customer.)

Comment thread sql2.txt
employee. Paste your SQL statement below.

INSERT INTO Orders
(CustomerID, EmployeeID, ShipperID)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There's no OrderDate here, but since the schema allows it and it's not a requirement in this exercise, I suppose this is fine. It's just a bit strange to have a single order with a NULL date.

Comment thread sql2.txt Outdated
0.50. Paste your SQL statement below.

UPDATE Products
SET Price = (Price * 0.50)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The requirement is:

the price increased by 0.50

but this does something else. Could you tell me what that is, and then fix it?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Oh man I halved the prices 😅 I'll fix it

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants