Skip to content

Complete Assignment two#2

Open
dzedze wants to merge 11 commits intomainfrom
assignment-two
Open

Complete Assignment two#2
dzedze wants to merge 11 commits intomainfrom
assignment-two

Conversation

@dzedze
Copy link
Copy Markdown
Owner

@dzedze dzedze commented Nov 23, 2025

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

To solve Assignment 2, Section 1:

  • Add 02_activities/assignments/Cohort_8/images/assignment2_section1_prompt1.pdf
  • Add 02_activities/assignments/Cohort_8/images/assignment2_section1_prompt2.pdf
  • Add 02_activities/assignments/Cohort_8/images/assignment2_section1_prompt3.pdf

To solve the rest of Assignment 2:

  • Update 02_activities/assignments/Cohort_8/assignment2.sql

What did you learn from the changes you have made?

  • How to design a simple bookstore database.
  • SQL: windowed functions, create, update, alter tables.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

n.a

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

  • Some errors: google for the corrected syntaxes.

How were these changes tested?

  • Look at values from 'Browse Data' or write some SELECT to see if changes are as expected.

A reference to a related issue in your repository (if applicable)

n.a

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@monzchan monzchan left a comment

Choose a reason for hiding this comment

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

Great work!

pq1.product_id,
pq1.quantity
FROM product_quantity pq1
WHERE market_date IS (
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I would suggest putting the table allias for clarity, and also IS is invalid for value comparison here. WHERE pq1.market_date = (SELECT MAX(...))

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