Skip to content

Comments

Assignment two#2

Open
AnaKanzaTariq wants to merge 6 commits intomainfrom
assignment-two
Open

Assignment two#2
AnaKanzaTariq wants to merge 6 commits intomainfrom
assignment-two

Conversation

@AnaKanzaTariq
Copy link
Owner

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

adding files

What did you learn from the changes you have made?

creating logical models

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

upload as png

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

great difficulty opening closing and saving sql files

How were these changes tested?

running queries

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

Checklist

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

Copy link

@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.

60/70

Section 4 is missing.

Review comments on Section 2,3:

Section 3: Cross join result set is incorrect. The query should return each vendor–product pair with the total price multiplied by the number of distinct customers, where price is defined as 5 × original_price.

SELECT vendor_name, product_name, sum(price)
FROM (
  SELECT DISTINCT
  vendor_name,
  product_name,
  5 * original_price as price

  FROM vendor_inventory vi 
  JOIN vendor v on v.vendor_id = vi.vendor_id
  JOIN product p on p.product_id = vi.product_id
  ) x
CROSS JOIN 
  (SELECT DISTINCT 
  customer_id
  FROM customer)
GROUP BY vendor_name, product_name

Section 3: Delete query: 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