Du blandar double precision
utdata från date_part()
med text '-'
. Det är inte vettigt för PostgreSQL. Du skulle behöva en explicit cast till text
. Men det finns ett mycket enklare sätt att göra allt detta:
startdate:=date_part('year',to_timestamp(NEW.date))
||'-'||date_part('month',to_timestamp(NEW.date))
||'-'||date_part('day',to_timestamp(NEW.date));
Använd istället:
startdate := to_char(NEW.date, 'YYYY-MM-DD');
Det här är inte heller meningsfullt:
EXECUTE 'CREATE TABLE $1 (
CHECK (date >= DATE $2 AND date < DATE $3 )
) INHERITS (pings)' USING quote_ident(tablename),startdate,enddate;
Du kan bara ange värden med USING
klausul. Läs manualen här . Försök istället:
EXECUTE 'CREATE TABLE ' || quote_ident(tablename) || ' (
CHECK ("date" >= ''' || startdate || ''' AND
"date" < ''' || enddate || '''))
INHERITS (ping)';
Eller ännu bättre, använd format()
. Se nedan.
Gilla också @a_horse svarade :Du måste sätta dina textvärden inom enkla citattecken.
Liknande här:
EXECUTE 'INSERT INTO $1 VALUES (NEW.*)' USING quote_ident(tablename);
Istället:
EXECUTE 'INSERT INTO ' || quote_ident(tablename) || ' VALUES ($1.*)'
USING NEW;
Relaterat svar:
Bortsett från:Även om "datum" är tillåtet för ett kolumnnamn i PostgreSQL är det en reserverat ord i varje SQL-standard . Döp inte din kolumn till "datum", det leder till förvirrande syntaxfel.
Fullständig fungerande demo
CREATE TABLE ping (ping_id integer, the_date date);
CREATE OR REPLACE FUNCTION trg_ping_partition()
RETURNS trigger AS
$func$
DECLARE
_tbl text := to_char(NEW.the_date, '"ping_"YYYY_DDD_') || NEW.ping_id;
BEGIN
IF NOT EXISTS (
SELECT 1
FROM pg_catalog.pg_class c
JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace
WHERE n.nspname = 'public' -- your schema
AND c.relname = _tbl
AND c.relkind = 'r') THEN
EXECUTE format('CREATE TABLE %I (CHECK (the_date >= %L AND
the_date < %L)) INHERITS (ping)'
, _tbl
, to_char(NEW.the_date, 'YYYY-MM-DD')
, to_char(NEW.the_date + 1, 'YYYY-MM-DD')
);
END IF;
EXECUTE 'INSERT INTO ' || quote_ident(_tbl) || ' VALUES ($1.*)'
USING NEW;
RETURN NULL;
END
$func$ LANGUAGE plpgsql SET search_path = public;
CREATE TRIGGER insbef
BEFORE INSERT ON ping
FOR EACH ROW EXECUTE PROCEDURE trg_ping_partition();
-
Uppdatering: Senare versioner av Postgres har mer eleganta sätt att kontrollera om en tabell finns:
-
to_char()
kan ta ettdate
som$1
. Det konverteras tilltimestamp
automatiskt.
Handboken om datum/tidsfunktioner . -
(Valfritt)
SET
search_path
för omfattningen av din funktion för att undvika missförhållanden med en ändradsearch_path
inställning. -
Flera andra förenklingar och förbättringar. Jämför koden.
Tester:
INSERT INTO ping VALUES (1, now()::date);
INSERT INTO ping VALUES (2, now()::date);
INSERT INTO ping VALUES (2, now()::date + 1);
INSERT INTO ping VALUES (2, now()::date + 1);