Skip to content

Assignment one#1

Open
alamfaiy wants to merge 8 commits intomainfrom
assignment-one
Open

Assignment one#1
alamfaiy wants to merge 8 commits intomainfrom
assignment-one

Conversation

@alamfaiy
Copy link
Copy Markdown
Owner

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

  • Adding a jpeg file containing the table relationships
  • Adding a "section 4" containing my essay-like response
  • Adding an sql file containing my answers for sections 2 and 3

What did you learn from the changes you have made?

  • I'm more comfortable with git
  • SQL is easier than I thought after working with matlab and python. It's quite straightforward. I particularly like the temp table feature.

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

  • Hmm, for the last question (I believe) I used a temp table. I wondered about how to answer that without a temp table?

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

  • I wrote it in the sql document, but I struggled with the second aggregate question using join.
  • This is what I wrote there: "In my mind, if I couldn't see the tables attached after join, they weren't joined. As a result I simply wasn't trusting that group by and having were working. However, I manually verified that this result was correct."

How were these changes tested?

  • Hmm, I tested the tables by making great use of the browse-data and constantly mini-checking using select by to see what each step was producing
  • For my changes in git, I kept double checking that my work was saved in the correct branch and showed up here after I did git push.

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

  • Mm, no issues off the top of my head.

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@kelichiu kelichiu left a comment

Choose a reason for hiding this comment

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

Point: 28/30

Section 1: ✅ Complete

Section 2: ✅ Complete

Section 3: ☑️ Partial

Aggregate Question 2

You are very close! You have the correct set of customers output in the result, and you have correctly joined the two tables together.

Todo:

  • You will need to display their total spending as well. Make an additional column from the calculation you have in the HAVING statement, give it a column name
  • You will need to sort the result. Sort the result by customer's last name, then first name
  • Not a must, but consider using table alias to shorten the table names in your queries

Date Question 2

You are also very close on this one! And I appreciate that you used temp table to complete this question.

Todo:

  • Check the correct number representation of "April"
  • Display additional columns of month and year

Section 4: ✅ Complete

Thank you Faiyza for the wonderful reflection! It's personal, thoughtful, and you have brought in valuable insights. It was a enjoyable read indeed.

Copy link
Copy Markdown

@kelichiu kelichiu left a comment

Choose a reason for hiding this comment

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

2nd assessment point: 30/30

Section 1: ✅ Complete

Section 2: ✅ Complete

Section 3: ✅ Complete

Section 4: ✅ Complete

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