Skip to content

Assignment two#2

Open
alamfaiy wants to merge 5 commits intomainfrom
assignment-two
Open

Assignment two#2
alamfaiy wants to merge 5 commits intomainfrom
assignment-two

Conversation

@alamfaiy
Copy link
Copy Markdown
Owner

@alamfaiy alamfaiy commented Feb 4, 2025

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

  • Added a figure with the bookstore architecture, completed section 4 and prompt 3 section 1, added an sql document with sections 2 and 3.

What did you learn from the changes you have made?

  • Making an architecture is harder than I thought...
  • I learned I like to abuse temp tables

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

  • Please see the comments in the sql file. In particular, I struggled with the union section.

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

  • I tend to struggle with cross join and functions like it (any join really) because it's difficult for me to visualize how the tables connect. I really needed to go back and forth in the browse data tab and I also drew some relationships out on paper.

How were these changes tested?

  • A lot of "select" statements lol

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

  • My inquires are written in the sql document

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@niyazmnazari niyazmnazari left a comment

Choose a reason for hiding this comment

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

Thank you for submitting assignment 2. Great job!
Point: 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