Skip to content

SQL - Assignment 2#2

Open
bolinfans wants to merge 2 commits intomainfrom
assignment-two
Open

SQL - Assignment 2#2
bolinfans wants to merge 2 commits intomainfrom
assignment-two

Conversation

@bolinfans
Copy link
Copy Markdown
Owner

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

  • completed assignment2.sql prompts
  • added analysis to assignment2.md of article and discussion on change tracking types
  • added diagrams for logic model for book store

What did you learn from the changes you have made?

  • I played around a lot with temptables and ctes for the sql coding section
  • for the logical model, I felt that there were a lot of ways to represent the store's data it it was hard selecting which one to use (i.e. dates in separate table, track book titles or ids, how to link books ordered to inventory, to sales, etc.)

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

  • playing more with ctes when implementing sql code than tamp tables
  • Definitely more ways to represent the logical model for the book store

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

  • logical model was challenging in that adding each new table needed a lot of consideration on how it would interact with other tables

How were these changes tested?

  • I manually verified some of the sql code, especially for calculating the total amount spent in -- Cross Join

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

Checklist

  • I can confirm that my changes are working as intended

Completed assignment2.md descriptions

added 2 diagrams in accordance with assignment2.md
Copy link
Copy Markdown

@anjali-deshpande-hub anjali-deshpande-hub left a comment

Choose a reason for hiding this comment

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

Very well done! 70/70

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