From a5f49cc080b4337c0023359a1d23d89b182b636d Mon Sep 17 00:00:00 2001 From: iliana etaoin Date: Wed, 22 May 2024 22:07:07 +0000 Subject: [PATCH] dbinit comment nit --- schema/crdb/dbinit.sql | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/schema/crdb/dbinit.sql b/schema/crdb/dbinit.sql index dbc59377e7..f2634cc061 100644 --- a/schema/crdb/dbinit.sql +++ b/schema/crdb/dbinit.sql @@ -3244,10 +3244,11 @@ CREATE TABLE IF NOT EXISTS omicron.public.blueprint ( -- CockroachDB settings managed by blueprints. -- - -- Settings are typed and not null; we use NULL in these columns to reflect - -- that blueprint execution should not modify these options. There is no - -- value that represents "please reset this setting to the default value"; - -- that is represented by the presence of the default value in that field. + -- We use NULL in these columns to reflect that blueprint execution should + -- not modify the option; we're able to do this because CockroachDB settings + -- require the value to be the correct type and not NULL. There is no value + -- that represents "please reset this setting to the default value"; that is + -- represented by the presence of the default value in that field. -- -- `cluster.preserve_downgrade_option` cockroachdb_setting_preserve_downgrade TEXT