Skip to content

Assignment two#2

Open
pradziie wants to merge 9 commits intomainfrom
assignment-two
Open

Assignment two#2
pradziie wants to merge 9 commits intomainfrom
assignment-two

Conversation

@pradziie
Copy link
Copy Markdown
Owner

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

Submitted Assignment 2. Section 3 was not fully completed, and it was too late to ask in office hours by that time.

What did you learn from the changes you have made?

I learned that if I don't know the answer, asking something like Chat GPT (even after the fact) will greatly hinder my learning.
I did learn the functions and some advanced SQL, though. Especially when it comes to theory as the classes were really good.

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

They are mentioned in the comments. A couple examples: compromise between type 1 and type 2 SCDs; section 2 part 3 also had me research after the fact.

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

A lot during SQL coding. I need to get used to it. Asking something like ChatGPT after the fact at some point made it even more challenging to understand the issues because it diminished my learning from trial and error. Had to go back to class recordings and my notes from classes which I kept, in order to try to solve this.

How were these changes tested?

in SQLite

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

Section 3 of assignment 2.

Checklist

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

Copy link
Copy Markdown

@DwardEE DwardEE left a comment

Choose a reason for hiding this comment

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

Good job!

The diagrams look good with no major problems.

Your Section 2 work is excellent: the COALESCE query is neatly done and handles NULLs correctly; the ROW_NUMBER approach is valid, and your step with reversing/CTEs shows strong understanding. Good catch using the COUNT OVER window, it’s much cleaner than the join version.

It's ok that you got outside help. The code looks good too. There are no issues with it.

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.

3 participants