Skip to content

Commit

Permalink
Merge pull request #244 from lsst/tickets/DM-45623-jeremym
Browse files Browse the repository at this point in the history
DM-45623: Make foreign key constraint names unique in cdb schemas
  • Loading branch information
JeremyMcCormick committed Aug 12, 2024
2 parents ce3f754 + 953c650 commit 56dc8ec
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 20 deletions.
14 changes: 7 additions & 7 deletions yml/cdb_latiss.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -302,13 +302,13 @@ tables:
- "#exposure_flexdata.obs_id"
- "#exposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_exposure_flexdata_obs_id
"@id": "#exposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be an Exposure exposure_id.
columns: ["#exposure_flexdata.obs_id"]
referencedColumns: ["#exposure.exposure_id"]
- name: fk_key
- name: fk_exposure_flexdata_key
"@id": "#exposure_flexdata.fk_key"
"@type": ForeignKey
description: Flex data key must be listed in the schema.
Expand Down Expand Up @@ -410,7 +410,7 @@ tables:
primaryKey:
- "#ccdexposure_camera.ccdexposure_id"
constraints:
- name: fk_ccdexposure_id
- name: fk_ccdexposure_camera_ccdexposure_id
"@id": "#ccdexposure_camera.fk_ccdexposure_id"
"@type": ForeignKey
description: exposure_id must be in the Exposure table.
Expand All @@ -437,13 +437,13 @@ tables:
- "#ccdexposure_flexdata.obs_id"
- "#ccdexposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_ccdexposure_flexdata_obs_id
"@id": "#ccdexposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be a CcdExposure ccdexposure_id.
columns: ["#ccdexposure_flexdata.obs_id"]
referencedColumns: ["#ccdexposure.ccdexposure_id"]
- name: fk_key
- name: fk_ccdexposure_flexdata_key
"@id": "#ccdexposure_flexdata.fk_key"
"@type": ForeignKey
description: Flex data key must be listed in the schema.
Expand Down Expand Up @@ -781,7 +781,7 @@ tables:
primaryKey:
- "#visit1_quicklook.visit_id"
constraints:
- name: fk_obs_id
- name: fk_visit1_quicklook_obs_id
"@id": "#visit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook visit_id must be an Exposure exposure_id.
Expand Down Expand Up @@ -948,7 +948,7 @@ tables:
primaryKey:
- "#ccdvisit1_quicklook.ccdvisit_id"
constraints:
- name: fk_obs_id
- name: fk_ccdvisit1_quicklook_obs_id
"@id": "#ccdvisit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook ccdvisit_id must be a CcdExposure ccdexposure_id.
Expand Down
14 changes: 7 additions & 7 deletions yml/cdb_lsstcomcam.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -290,13 +290,13 @@ tables:
- "#exposure_flexdata.obs_id"
- "#exposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_exposure_flexdata_obs_id
"@id": "#exposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be an Exposure exposure_id.
columns: ["#exposure_flexdata.obs_id"]
referencedColumns: ["#exposure.exposure_id"]
- name: fk_key
- name: fk_exposure_flexdata_key
"@id": "#exposure_flexdata.fk_key"
"@type": ForeignKey
description: Flex data key must be listed in the schema.
Expand Down Expand Up @@ -398,7 +398,7 @@ tables:
primaryKey:
- "#ccdexposure_camera.ccdexposure_id"
constraints:
- name: fk_ccdexposure_id
- name: fk_ccdexposure_camera_ccdexposure_id
"@id": "#ccdexposure_camera.fk_ccdexposure_id"
"@type": ForeignKey
description: exposure_id must be in the Exposure table.
Expand All @@ -425,13 +425,13 @@ tables:
- "#ccdexposure_flexdata.obs_id"
- "#ccdexposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_ccdexposure_flexdata_obs_id
"@id": "#ccdexposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be a CcdExposure ccdexposure_id.
columns: ["#ccdexposure_flexdata.obs_id"]
referencedColumns: ["#ccdexposure.ccdexposure_id"]
- name: fk_key
- name: fk_ccdexposure_flexdata_key
"@id": "#ccdexposure_flexdata.fk_key"
"@type": ForeignKey
description: Flex data key must be listed in the schema.
Expand Down Expand Up @@ -761,7 +761,7 @@ tables:
primaryKey:
- "#visit1_quicklook.visit_id"
constraints:
- name: fk_obs_id
- name: fk_visit1_quicklook_obs_id
"@id": "#visit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook visit_id must be an Exposure exposure_id.
Expand Down Expand Up @@ -1176,7 +1176,7 @@ tables:
primaryKey:
- "#ccdvisit1_quicklook.ccdvisit_id"
constraints:
- name: fk_obs_id
- name: fk_ccdvisit1_quicklook_obs_id
"@id": "#ccdvisit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook ccdvisit_id must be a CcdExposure ccdexposure_id.
Expand Down
12 changes: 6 additions & 6 deletions yml/cdb_lsstcomcamsim.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -290,13 +290,13 @@ tables:
- "#exposure_flexdata.obs_id"
- "#exposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_exposure_flexdata_obs_id
"@id": "#exposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be an Exposure exposure_id.
columns: ["#exposure_flexdata.obs_id"]
referencedColumns: ["#exposure.exposure_id"]
- name: fk_key
- name: fk_exposure_flexdata_key
"@id": "#exposure_flexdata.fk_key"
"@type": ForeignKey
description: Flex data key must be listed in the schema.
Expand Down Expand Up @@ -398,7 +398,7 @@ tables:
primaryKey:
- "#ccdexposure_camera.ccdexposure_id"
constraints:
- name: fk_ccdexposure_id
- name: fk_ccdexposure_camera_ccdexposure_id
"@id": "#ccdexposure_camera.fk_ccdexposure_id"
"@type": ForeignKey
description: exposure_id must be in the Exposure table.
Expand All @@ -425,7 +425,7 @@ tables:
- "#ccdexposure_flexdata.obs_id"
- "#ccdexposure_flexdata.key"
constraints:
- name: fk_obs_id
- name: fk_ccdexposure_flexdata_obs_id
"@id": "#ccdexposure_flexdata.fk_obs_id"
"@type": ForeignKey
description: Flex data obs_id must be a CcdExposure ccdexposure_id.
Expand Down Expand Up @@ -761,7 +761,7 @@ tables:
primaryKey:
- "#visit1_quicklook.visit_id"
constraints:
- name: fk_obs_id
- name: fk_visit1_quicklook_obs_id
"@id": "#visit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook visit_id must be an Exposure exposure_id.
Expand Down Expand Up @@ -1176,7 +1176,7 @@ tables:
primaryKey:
- "#ccdvisit1_quicklook.ccdvisit_id"
constraints:
- name: fk_obs_id
- name: fk_ccdvisit1_quicklook_obs_id
"@id": "#ccdvisit1_quicklook.fk_obs_id"
"@type": ForeignKey
description: Quicklook ccdvisit_id must be a CcdExposure ccdexposure_id.
Expand Down

0 comments on commit 56dc8ec

Please sign in to comment.