Skip to content

UofT-DSI | SQL - Assignment 2 - #503

Closed
mouweili wants to merge 2 commits into
UofT-DSI:mainfrom
mouweili:assignment-two
Closed

mouweili wants to merge 2 commits into
UofT-DSI:mainfrom
mouweili:assignment-two

Conversation

@mouweili

@mouweili mouweili commented Nov 6, 2026

Copy link
Copy Markdown

What changes are you trying to make?

I completed Assignment 2: designed a bookstore logical model (with and without employee shifts), answered the Type 1 vs Type 2 slowly changing dimension prompt, wrote the SQL for Sections 2 and 3 in assignment2.sql, and wrote the Section 4 ethics response in Assignment2.md. This branch was created from assignment-one, so the Assignment 1 files also appear in the diff.

What did you learn from the changes you have made?

I learned how to use COALESCE, window functions, string functions, UNION, CROSS JOIN, and INSERT/DELETE/UPDATE, and how to design a model with a date table and slowly changing dimensions.

Was there another approach you were thinking about making?

For the most recent visit query, I considered ROW_NUMBER versus DENSE_RANK. I used DENSE_RANK so multiple purchases on the same market date share the same visit number.

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

Keeping each query separate and handling tables that already existed when re-running statements. I fixed these by testing queries one at a time.

How were these changes tested?

I ran each query in DB Browser for SQLite and checked row counts and column values.

Checklist

  • I can confirm that my changes are working as intended

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

This pull request was made to the wrong repository. If you are a participant, please close it and open it in your own fork instead. Refer to the Assignment Submission Guide for detailed instructions.

@github-actions github-actions Bot closed this Nov 6, 2026
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.

1 participant