Drop unused columns from lookup tables (medplum#4294)

awsm-eng · Apr 3, 2024 · 4306ebc · 4306ebc
1 parent 873f6c1
commit 4306ebc
Show file tree

Hide file tree

Showing 3 changed files with 192 additions and 1 deletion.
diff --git a/packages/server/src/fhir/operations/expunge.test.ts b/packages/server/src/fhir/operations/expunge.test.ts
@@ -193,6 +193,9 @@ async function existsInDatabase(tableName: string, id: string | undefined): Prom
 }
 
 async function existsInLookupTable(tableName: string, id: string | undefined): Promise<boolean> {
-  const rows = await new SelectQuery(tableName).column('id').where('resourceId', '=', id).execute(getDatabasePool());
+  const rows = await new SelectQuery(tableName)
+    .column('resourceId')
+    .where('resourceId', '=', id)
+    .execute(getDatabasePool());
   return rows.length > 0;
 }
diff --git a/packages/server/src/migrations/schema/index.ts b/packages/server/src/migrations/schema/index.ts
@@ -72,3 +72,4 @@ export * as v63 from './v63';
 export * as v64 from './v64';
 export * as v65 from './v65';
 export * as v66 from './v66';
+export * as v67 from './v67';
diff --git a/packages/server/src/migrations/schema/v67.ts b/packages/server/src/migrations/schema/v67.ts
@@ -0,0 +1,187 @@
+/*
+ * Generated by @medplum/generator
+ * Do not edit manually.
+ */
+
+import { PoolClient } from 'pg';
+
+const resourceTypes = [
+  'AccessPolicy',
+  'Account',
+  'ActivityDefinition',
+  'AdverseEvent',
+  'Agent',
+  'AllergyIntolerance',
+  'Appointment',
+  'AppointmentResponse',
+  'AsyncJob',
+  'AuditEvent',
+  'Basic',
+  'Binary',
+  'BiologicallyDerivedProduct',
+  'BodyStructure',
+  'Bot',
+  'BulkDataExport',
+  'Bundle',
+  'CapabilityStatement',
+  'CarePlan',
+  'CareTeam',
+  'CatalogEntry',
+  'ChargeItem',
+  'ChargeItemDefinition',
+  'Claim',
+  'ClaimResponse',
+  'ClientApplication',
+  'ClinicalImpression',
+  'CodeSystem',
+  'Communication',
+  'CommunicationRequest',
+  'CompartmentDefinition',
+  'Composition',
+  'ConceptMap',
+  'Condition',
+  'Consent',
+  'Contract',
+  'Coverage',
+  'CoverageEligibilityRequest',
+  'CoverageEligibilityResponse',
+  'DetectedIssue',
+  'Device',
+  'DeviceDefinition',
+  'DeviceMetric',
+  'DeviceRequest',
+  'DeviceUseStatement',
+  'DiagnosticReport',
+  'DocumentManifest',
+  'DocumentReference',
+  'DomainConfiguration',
+  'EffectEvidenceSynthesis',
+  'Encounter',
+  'Endpoint',
+  'EnrollmentRequest',
+  'EnrollmentResponse',
+  'EpisodeOfCare',
+  'EventDefinition',
+  'Evidence',
+  'EvidenceVariable',
+  'ExampleScenario',
+  'ExplanationOfBenefit',
+  'FamilyMemberHistory',
+  'Flag',
+  'Goal',
+  'GraphDefinition',
+  'Group',
+  'GuidanceResponse',
+  'HealthcareService',
+  'ImagingStudy',
+  'Immunization',
+  'ImmunizationEvaluation',
+  'ImmunizationRecommendation',
+  'ImplementationGuide',
+  'InsurancePlan',
+  'Invoice',
+  'JsonWebKey',
+  'Library',
+  'Linkage',
+  'List',
+  'Location',
+  'Login',
+  'Measure',
+  'MeasureReport',
+  'Media',
+  'Medication',
+  'MedicationAdministration',
+  'MedicationDispense',
+  'MedicationKnowledge',
+  'MedicationRequest',
+  'MedicationStatement',
+  'MedicinalProduct',
+  'MedicinalProductAuthorization',
+  'MedicinalProductContraindication',
+  'MedicinalProductIndication',
+  'MedicinalProductIngredient',
+  'MedicinalProductInteraction',
+  'MedicinalProductManufactured',
+  'MedicinalProductPackaged',
+  'MedicinalProductPharmaceutical',
+  'MedicinalProductUndesirableEffect',
+  'MessageDefinition',
+  'MessageHeader',
+  'MolecularSequence',
+  'NamingSystem',
+  'NutritionOrder',
+  'Observation',
+  'ObservationDefinition',
+  'OperationDefinition',
+  'OperationOutcome',
+  'Organization',
+  'OrganizationAffiliation',
+  'Parameters',
+  'PasswordChangeRequest',
+  'Patient',
+  'PaymentNotice',
+  'PaymentReconciliation',
+  'Person',
+  'PlanDefinition',
+  'Practitioner',
+  'PractitionerRole',
+  'Procedure',
+  'Project',
+  'ProjectMembership',
+  'Provenance',
+  'Questionnaire',
+  'QuestionnaireResponse',
+  'RelatedPerson',
+  'RequestGroup',
+  'ResearchDefinition',
+  'ResearchElementDefinition',
+  'ResearchStudy',
+  'ResearchSubject',
+  'RiskAssessment',
+  'RiskEvidenceSynthesis',
+  'Schedule',
+  'SearchParameter',
+  'ServiceRequest',
+  'Slot',
+  'SmartAppLaunch',
+  'Specimen',
+  'SpecimenDefinition',
+  'StructureDefinition',
+  'StructureMap',
+  'Subscription',
+  'SubscriptionStatus',
+  'Substance',
+  'SubstanceNucleicAcid',
+  'SubstancePolymer',
+  'SubstanceProtein',
+  'SubstanceReferenceInformation',
+  'SubstanceSourceMaterial',
+  'SubstanceSpecification',
+  'SupplyDelivery',
+  'SupplyRequest',
+  'Task',
+  'TerminologyCapabilities',
+  'TestReport',
+  'TestScript',
+  'User',
+  'UserConfiguration',
+  'ValueSet',
+  'VerificationResult',
+  'VisionPrescription',
+];
+
+export async function run(client: PoolClient): Promise<void> {
+  await client.query('ALTER TABLE "Address" DROP COLUMN "id"');
+  await client.query('ALTER TABLE "Address" DROP COLUMN "index"');
+  await client.query('ALTER TABLE "Address" DROP COLUMN "content"');
+
+  await client.query('ALTER TABLE "HumanName" DROP COLUMN "id"');
+  await client.query('ALTER TABLE "HumanName" DROP COLUMN "index"');
+  await client.query('ALTER TABLE "HumanName" DROP COLUMN "content"');
+
+  await client.query('ALTER TABLE "ValueSetElement" DROP COLUMN "id"');
+
+  for (const resourceType of resourceTypes) {
+    await client.query(`ALTER TABLE "${resourceType}_Token" DROP COLUMN "index"`);
+  }
+}