mtucker

This is a set of ‘statuses’ that pull request comments must be marked with to help explain intent.

• Required: This must be fixed before merge.
• Nice to have: Would be great to fix eventually.
• Personal Preference: I would do this, but you don’t have to.
• Question: Point of clarification?
• Potential Alternative: Presenting an alternative approach in order to invite discussion.
• Suggested Change: I have pushed a set of changes that I want to call out.

^{Modified from Scaling Square Register on objc.io}

mtucker

DROP TABLE if exists d_date;

CREATE TABLE d_date
(
  date_dim_id              INT NOT NULL,
  date_actual              DATE NOT NULL,
  epoch                    BIGINT NOT NULL,
  day_suffix               VARCHAR(4) NOT NULL,
  day_name                 VARCHAR(9) NOT NULL,
  day_of_week              INT NOT NULL,