Skip to content

Assignment two#2

Open
SereneMousaoui wants to merge 12 commits intomainfrom
assignment-two
Open

Assignment two#2
SereneMousaoui wants to merge 12 commits intomainfrom
assignment-two

Conversation

@SereneMousaoui
Copy link
Copy Markdown
Owner

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

Adding code, markdown, and logical models

What did you learn from the changes you have made?

I learned a lot about sub-querying, improving logical models to minimize many-to-many relationships, and reflecting on AI and ethics.

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

There were many different approaches to the possible queries for the assignment 2 SQL file. I tried to incorporate different techniques intentionally. For one query, I made a temp table, and for another, a CTE using 'with' queries. I appreciate the opportunity to practice everything we learned in class with this.

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

YES! Especially for the last few queries. Getting it to output what I wanted and finding out where to start. I used a lot of googling and asking for advice from my SQL-professional friends.

How were these changes tested?

I tested directly in SQL

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

NA

Checklist

  • [X ] I can confirm that my changes are working as intended

@SereneMousaoui
Copy link
Copy Markdown
Owner Author

Please note that there are 2 .png files for the logical models. Bookstore_logical_model_SM.png is for Section 1, Prompt 1. Bookstore_logical_model_prompt2_SM.png is for Section 1, Prompt 2.

@khsergvl
Copy link
Copy Markdown

Great attempt, reviewed. Try to play with git flows to better understand how does it work.

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