Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bugs in definition of job fact tables. #1349

Merged
merged 3 commits into from
Jun 25, 2020
Merged

Conversation

jpwhite4
Copy link
Member

Dimensions can never be null.
Update typo in comment fields.

Dimensions can never be null.
Update typo in comment fields.
@jpwhite4 jpwhite4 requested a review from jtpalmer June 24, 2020 13:43
@jpwhite4 jpwhite4 added the bug Bugfixes label Jun 24, 2020
@jpwhite4 jpwhite4 added this to the 9.0.0 milestone Jun 24, 2020
plessbd
plessbd previously approved these changes Jun 24, 2020
jtpalmer
jtpalmer previously approved these changes Jun 24, 2020
configuration/etl/etl_tables.d/jobs/xdw/jobfact_by_.json Outdated Show resolved Hide resolved
@jtpalmer
Copy link
Contributor

You might also consider fixing configuration/etl/etl_tables.d/cloud_common/cloudfact_by_.json since it has the same issues.

Co-authored-by: Jeffrey T. Palmer <jeffrey.t.palmer@gmail.com>
@jpwhite4 jpwhite4 dismissed stale reviews from jtpalmer and plessbd via a4660e8 June 24, 2020 14:27
Co-authored-by: Jeffrey T. Palmer <jeffrey.t.palmer@gmail.com>
@jpwhite4 jpwhite4 merged commit 0486f22 into ubccr:xdmod9.0 Jun 25, 2020
@jpwhite4 jpwhite4 deleted the factDIM branch June 25, 2020 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugfixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants