-
Notifications
You must be signed in to change notification settings - Fork 0
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
Max David Metelus : Open Pull Request :Work In Progress. #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, great job!
where PostalCode = 1010; | ||
|
||
select * from customers | ||
where PostalCode = '1010'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a duplicate here.
where SupplierID = 11; | ||
|
||
select * from Suppliers | ||
where SupplierID = 11; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here as well
select orders.CustomerID, customers.CustomerName, count(customers.customerID) | ||
from Orders | ||
inner join Customers | ||
on Orders.CustomerID=Customers.CustomerID; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using aliases here would make this a good bit easier/shorter. This also looks a little off, but close!
Open Pull Request