Skip to content

Commit

Permalink
Set datetime fields to nullable rather than required
Browse files Browse the repository at this point in the history
 * As defined by OMOP spec (see https://github.com/OHDSI/CommonDataModel)
 * Datetimes in observation_period will likely be removed (see OHDSI/CommonDataModel#64)
  • Loading branch information
mark-velez committed Jul 1, 2017
1 parent 97519dd commit 61bae0a
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion tools/bq/schemas/condition_occurrence.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
{
"type": "timestamp",
"name": "condition_start_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "date",
Expand Down
2 changes: 1 addition & 1 deletion tools/bq/schemas/device_exposure.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
{
"type": "timestamp",
"name": "device_exposure_start_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "date",
Expand Down
2 changes: 1 addition & 1 deletion tools/bq/schemas/drug_exposure.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
{
"type": "timestamp",
"name": "drug_exposure_start_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "date",
Expand Down
4 changes: 2 additions & 2 deletions tools/bq/schemas/observation_period.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
{
"type": "timestamp",
"name": "observation_period_start_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "date",
Expand All @@ -27,7 +27,7 @@
{
"type": "timestamp",
"name": "observation_period_end_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "integer",
Expand Down
2 changes: 1 addition & 1 deletion tools/bq/schemas/procedure_occurrence.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
{
"type": "timestamp",
"name": "procedure_datetime",
"mode": "required"
"mode": "nullable"
},
{
"type": "integer",
Expand Down

0 comments on commit 61bae0a

Please sign in to comment.