Conversation
efantinatti
left a comment
There was a problem hiding this comment.
Dear Anna Veremchuk
Session 1:
1 - Partial. Almost good. Check the date table connection requirements to what is asked in the question.
Session 2:
2 - Partial.
-- Windowed Functions - Q3: Query is not correct. You can think in a simpler approach with COUNT() and Partition.
Session 3:
3 - Partial.
-- DELETE - Q1: The WHERE clause condition is hardcoded. Think of a solution that it might process it without hardcoding a value. HINT. Think of minimal values,. Which function could find the minal value?
Final:
4 - Check the aforementioned poins prior merging your feature branch 'assignment-two' to 'main'.
Mark: 50 points.
|
Dear Ernani Fantinatti @efantinatti , Thank you for checking and your comments! "1 - Partial. Almost good. Check the date table connection requirements to what is asked in the question." Could you please add more details why points were reduced for this ? The only thing that I found in task text in regard to date table connection is the following: "2 - Partial. Could you please specify why this query is not correct? Task: /* 3. Using a COUNT() window function, include a value along with each row of the Query in submission: Field total_times_product_id_purchased_by_customer shows number of times product was purchased by customer for every row in customer_purchases table. I have manually checked on two customers and two products and quantities are correct, but it is possible that I am missing something... "Session 3: Thank you for the hint, it should be fixed now: BR, |
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist