Skip to content

Assignment two#2

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

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

Conversation

@Charlotte11b
Copy link
Copy Markdown
Owner

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

Submitting assignments.

What did you learn from the changes you have made?

SQL

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

N/A

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

The questions are more challenging this time.

How were these changes tested?

I input every SQL command into DB browswer for SQLite to execute and test the results.

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

N/A

Checklist

  • I can confirm that my changes are working as intended

Assignment1.md and assignment1.sql.
Added Logical Data Model.png and farmersmarket.sql.
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 (68/70)

Just a few comments:
Section 3: Cross join - DISTINCT missing in vendor_products query at line 194.

Section 3: Delete: The requirement is to delete the older record for the product added.

You can use MIN(snapshot_timestamp) function. Using the WITH clause and MIN function, identify the earliest snapshot for the product_id and then delete the row that matches that product_id and timestamp.

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