Skip to content

Updated with my answers for Homework 5#5

Open
amlloren wants to merge 1 commit intomainfrom
homework-5
Open

Updated with my answers for Homework 5#5
amlloren wants to merge 1 commit intomainfrom
homework-5

Conversation

@amlloren
Copy link
Copy Markdown
Owner

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

I added my answers to Homework 5

What did you learn from the changes you have made?

I learned how to manipulate and manage data in SQL using various commands such as ALTER TABLE, UPDATE, and COALESCE. Specifically, I learned how to add new columns to existing tables and update them based on complex queries that involve subqueries and joins.

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

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

One challenge was ensuring that the subquery for retrieving the last quantity accurately linked to the correct product without duplicating or missing any records. I overcame this by carefully structuring the subquery to filter by both product_id and market_date, ensuring that I retrieved the most recent quantity for each product. Testing each part of the query incrementally helped me identify any issues early on.

How were these changes tested?

I tested the changes by running SELECT queries on the product_units table after the UPDATE operation to verify that the current_quantity values were populated correctly. I also cross-referenced the results with the vendor_inventory table to ensure that the latest quantities matched the expected values. Additionally, I checked edge cases where products might not have any quantity to ensure the COALESCE function worked as intended.

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

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@amanda-ng518 amanda-ng518 left a comment

Choose a reason for hiding this comment

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

HW5: 7/8

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