Skip to content

Commit

Permalink
Merge pull request #1349 from jpwhite4/factDIM
Browse files Browse the repository at this point in the history
Fix bugs in definition of job fact tables.
  • Loading branch information
jpwhite4 authored Jun 25, 2020
2 parents 85389ab + 77e9b2d commit 0486f22
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions configuration/etl/etl_tables.d/jobs/xdw/jobfact_by_.json
Original file line number Diff line number Diff line change
Expand Up @@ -157,8 +157,8 @@
{
"name": "processorbucket_id",
"type": "int(4)",
"comment": "FACT: Pre-determined processor bucket sizes. References processorbucket.id",
"nullable": true
"comment": "DIMENSION: Pre-determined processor bucket sizes. References processor_buckets.id",
"nullable": false
},
{
"name": "gpu_count",
Expand All @@ -169,7 +169,7 @@
{
"name": "gpubucket_id",
"type": "int(11)",
"nullable": true,
"nullable": false,
"comment": "DIMENSION: Pre-determined GPU bucket sizes. References gpu_buckets.id"
},
{
Expand Down
6 changes: 3 additions & 3 deletions configuration/etl/etl_tables.d/jobs/xdw/jobfact_by_day.json
Original file line number Diff line number Diff line change
Expand Up @@ -163,8 +163,8 @@
{
"name": "processorbucket_id",
"type": "int(4)",
"comment": "FACT: Pre-determined processor bucket sizes. References processorbucket.id",
"nullable": true
"comment": "DIMENSION: Pre-determined processor bucket sizes. References processor_buckets.id",
"nullable": false
},
{
"name": "gpu_count",
Expand All @@ -175,7 +175,7 @@
{
"name": "gpubucket_id",
"type": "int(11)",
"nullable": true,
"nullable": false,
"comment": "DIMENSION: Pre-determined GPU bucket sizes. References gpu_buckets.id"
},
{
Expand Down

0 comments on commit 0486f22

Please sign in to comment.