Skip to content

Assignment two#2

Open
MankitWong wants to merge 3 commits intomainfrom
assignment-two
Open

Assignment two#2
MankitWong wants to merge 3 commits intomainfrom
assignment-two

Conversation

@MankitWong
Copy link
Copy Markdown
Owner

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

modified the Assignment2.md for answering the type1 and type2 SCD
added 2 png for ERD
modified the Assignment2.sql for SQL questions

What did you learn from the changes you have made?

more familiar with the windowed functions

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

was using with clause on most of the SQL Questions, but probably can use temporary tables
some of the Question that use the windowed functions that probably can use max and min inline query to get the same reason

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

no big challenges. it is more like google for the syntax of the SQL

How were these changes tested?

the changes was testing section by section like test each of the with clause first and cross check manually with list out the reason if their output as expected.

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

I guess the problem I branch out from assignment-one branch, so the pull request from assignment-two to main. it also included assignment-one changes.

Checklist

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

MankitWong and others added 3 commits November 14, 2025 22:45
Updated SQL query to use COALESCE for handling NULL values in product details.
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. All sections are correct.
Mark: 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