Do not quote boolean default values inside bucardo.schema

diff --git a/bucardo.schema b/bucardo.schema
index c3a0c3497e4e5a7402975ca2aa27d237fbfb080b..0d8ccb4e3814203cbb7f16b8bfa4ace4dede4b09 100644 (file)
--- a/bucardo.schema
+++ b/bucardo.schema
@@ -200,7 +200,7 @@ CREATE TABLE bucardo.db (
   dbservice            TEXT            NULL DEFAULT '',
   pgpass               TEXT            NULL,      -- local file with connection info same as pgpass
   status               TEXT        NOT NULL DEFAULT 'active',
-  server_side_prepares BOOLEAN     NOT NULL DEFAULT 'true',
+  server_side_prepares BOOLEAN     NOT NULL DEFAULT true,
   cdate                TIMESTAMPTZ NOT NULL DEFAULT now()
 );
 COMMENT ON TABLE bucardo.db IS $$Holds information about each database used in replication$$;
@@ -264,16 +264,16 @@ CREATE TABLE bucardo.goat (
   pkey               TEXT            NULL,
   qpkey              TEXT            NULL,
   pkeytype           TEXT            NULL,
-  has_delta          BOOLEAN     NOT NULL DEFAULT 'false',
+  has_delta          BOOLEAN     NOT NULL DEFAULT false,
   autokick           BOOLEAN         NULL,                 -- overrides sync-level autokick
   conflict_strategy  TEXT            NULL,
   makedelta          TEXT            NULL,
   rebuild_index      SMALLINT        NULL DEFAULT 0,       -- overrides sync-level rebuild_index
-  ghost              BOOLEAN     NOT NULL DEFAULT 'false', -- only drop triggers, do not replicate
-  analyze_after_copy BOOLEAN     NOT NULL DEFAULT 'true',
-  vacuum_after_copy  BOOLEAN     NOT NULL DEFAULT 'true',
-  strict_checking    BOOLEAN     NOT NULL DEFAULT 'true',
-  delta_bypass       BOOLEAN     NOT NULL DEFAULT 'false',
+  ghost              BOOLEAN     NOT NULL DEFAULT false, -- only drop triggers, do not replicate
+  analyze_after_copy BOOLEAN     NOT NULL DEFAULT true,
+  vacuum_after_copy  BOOLEAN     NOT NULL DEFAULT true,
+  strict_checking    BOOLEAN     NOT NULL DEFAULT true,
+  delta_bypass       BOOLEAN     NOT NULL DEFAULT false,
   delta_bypass_min   BIGINT          NULL,
   delta_bypass_count BIGINT          NULL,
   delta_bypass_percent   SMALLINT    NULL,
@@ -397,8 +397,8 @@ CREATE TABLE bucardo.sync (
                        CONSTRAINT sync_herd_fk FOREIGN KEY (herd) REFERENCES bucardo.herd(name) ON UPDATE CASCADE ON DELETE RESTRICT,
   dbs                TEXT            NULL,
                        CONSTRAINT sync_dbs_fk FOREIGN KEY (dbs) REFERENCES bucardo.dbgroup(name) ON UPDATE CASCADE ON DELETE RESTRICT,
-  stayalive          BOOLEAN     NOT NULL DEFAULT 'true', -- Does the sync controller stay connected?
-  kidsalive          BOOLEAN     NOT NULL DEFAULT 'true', -- Do the children stay connected?
+  stayalive          BOOLEAN     NOT NULL DEFAULT true, -- Does the sync controller stay connected?
+  kidsalive          BOOLEAN     NOT NULL DEFAULT true, -- Do the children stay connected?
   conflict_strategy  TEXT        NOT NULL DEFAULT 'bucardo_latest',
   copyextra          TEXT        NOT NULL DEFAULT '',  -- e.g. WITH OIDS
   deletemethod       TEXT        NOT NULL DEFAULT 'delete',
@@ -407,12 +407,12 @@ CREATE TABLE bucardo.sync (
   status             TEXT        NOT NULL DEFAULT 'active',  -- Possibly CHECK / FK ('stopped','paused','b0rken')
   rebuild_index      SMALLINT    NOT NULL DEFAULT 0,     -- Load without indexes and then REINDEX table
   priority           SMALLINT    NOT NULL DEFAULT 0,     -- Higher is better
-  analyze_after_copy BOOLEAN     NOT NULL DEFAULT 'true',
-  vacuum_after_copy  BOOLEAN     NOT NULL DEFAULT 'false',
-  strict_checking    BOOLEAN     NOT NULL DEFAULT 'true',
+  analyze_after_copy BOOLEAN     NOT NULL DEFAULT true,
+  vacuum_after_copy  BOOLEAN     NOT NULL DEFAULT false,
+  strict_checking    BOOLEAN     NOT NULL DEFAULT true,
   overdue            INTERVAL    NOT NULL DEFAULT '0 seconds'::interval,
   expired            INTERVAL    NOT NULL DEFAULT '0 seconds'::interval,
-  track_rates        BOOLEAN     NOT NULL DEFAULT 'false',
+  track_rates        BOOLEAN     NOT NULL DEFAULT false,
   onetimecopy        SMALLINT    NOT NULL DEFAULT 0,
   lifetime           INTERVAL        NULL,                   -- force controller and kids to restart
   maxkicks           INTEGER     NOT NULL DEFAULT 0,         -- force controller and kids to restart
@@ -441,7 +441,7 @@ CREATE TABLE bucardo.customcode (
   name      TEXT        NOT NULL UNIQUE,
   about     TEXT            NULL,
   whenrun   TEXT        NOT NULL,
-  getdbh    BOOLEAN     NOT NULL DEFAULT 'true',
+  getdbh    BOOLEAN     NOT NULL DEFAULT true,
   src_code  TEXT        NOT NULL,
   status    TEXT        NOT NULL DEFAULT 'active',
   priority  SMALLINT    NOT NULL DEFAULT 0,    
@@ -470,7 +470,7 @@ CREATE TABLE bucardo.customcode_map (
              CONSTRAINT customcode_map_sync_fk FOREIGN KEY (sync) REFERENCES bucardo.sync(name) ON UPDATE CASCADE ON DELETE SET NULL,
   goat     INTEGER         NULL,
              CONSTRAINT customcode_map_goat_fk FOREIGN KEY (goat) REFERENCES bucardo.goat(id) ON DELETE SET NULL,
-  active   BOOLEAN     NOT NULL DEFAULT 'true',
+  active   BOOLEAN     NOT NULL DEFAULT true,
   priority SMALLINT    NOT NULL DEFAULT 0,
   cdate    TIMESTAMPTZ NOT NULL DEFAULT now()
 );
@@ -556,9 +556,9 @@ CREATE TABLE bucardo.syncrun (
   conflicts BIGINT       NOT NULL DEFAULT 0,
   started   TIMESTAMPTZ  NOT NULL DEFAULT now(),
   ended     TIMESTAMPTZ      NULL,
-  lastgood  BOOLEAN      NOT NULL DEFAULT 'false',
-  lastbad   BOOLEAN      NOT NULL DEFAULT 'false',
-  lastempty BOOLEAN      NOT NULL DEFAULT 'false',
+  lastgood  BOOLEAN      NOT NULL DEFAULT false,
+  lastbad   BOOLEAN      NOT NULL DEFAULT false,
+  lastempty BOOLEAN      NOT NULL DEFAULT false,
   details   TEXT             NULL,
   status    TEXT             NULL
 );