Merge pull request #960 from NASA-AMMOS/fix/make-constraints-descript…

…ion-optional

Make `Description` optional across the DB

deployment/hasura/migrations/AerieMerlin/15_defintion_consistency/down.sql

@@ -0,0 +1,7 @@
+alter table public.simulation_template
+  alter column description drop default;
+
+alter table public."constraint"
+  alter column description drop default;
+
+call migrations.mark_migration_rolled_back('15');

deployment/hasura/migrations/AerieMerlin/15_defintion_consistency/up.sql

@@ -0,0 +1,7 @@
+alter table public."constraint"
+  alter column description set default '';
+
+alter table public.simulation_template
+  alter column description set default '';
+
+call migrations.mark_migration_applied('15');

deployment/hasura/migrations/AerieScheduler/7_definition_consistency/down.sql

@@ -0,0 +1,9 @@
+alter table scheduling_condition
+  alter column description drop default,
+  alter column description drop not null;
+
+alter table scheduling_goal
+  alter column description drop default,
+  alter column description drop not null;
+
+call migrations.mark_migration_rolled_back('7');

deployment/hasura/migrations/AerieScheduler/7_definition_consistency/up.sql

@@ -0,0 +1,9 @@
+alter table scheduling_goal
+  alter column description set default '',
+  alter column description set not null;
+
+alter table scheduling_condition
+  alter column description set default '',
+  alter column description set not null;
+
+call migrations.mark_migration_applied('7');

merlin-server/sql/merlin/applied_migrations.sql

@@ -17,3 +17,4 @@ call migrations.mark_migration_applied('11');
 call migrations.mark_migration_applied('12');
 call migrations.mark_migration_applied('13');
 call migrations.mark_migration_applied('14');
+call migrations.mark_migration_applied('15');

merlin-server/sql/merlin/tables/constraint.sql

@@ -2,7 +2,7 @@ create table "constraint" (
   id integer generated always as identity,
 
   name text not null,
-  description text not null,
+  description text not null default '',
   definition text not null,
   tags text[] default '{}',
 

merlin-server/sql/merlin/tables/simulation_template.sql

@@ -3,7 +3,7 @@ create table simulation_template (
   revision integer not null default 0,
 
   model_id integer not null,
-  description text not null,
+  description text not null default '',
   arguments merlin_argument_set not null,
   owner text not null default '',
 

scheduler-server/sql/scheduler/applied_migrations.sql

@@ -9,3 +9,4 @@ call migrations.mark_migration_applied('3');
 call migrations.mark_migration_applied('4');
 call migrations.mark_migration_applied('5');
 call migrations.mark_migration_applied('6');
+call migrations.mark_migration_applied('7');

scheduler-server/sql/scheduler/tables/scheduling_condition.sql

@@ -5,7 +5,7 @@ create table scheduling_condition (
   definition text not null,
 
   model_id integer not null,
-  description text null,
+  description text not null default '',
   author text null,
   last_modified_by text null,
   created_date timestamptz not null default now(),

scheduler-server/sql/scheduler/tables/scheduling_goal.sql

@@ -6,7 +6,7 @@ create table scheduling_goal (
   tags text[] not null default '{}',
 
   model_id integer not null,
-  description text null,
+  description text not null default '',
   author text null,
   last_modified_by text null,
   created_date timestamptz not null default now(),