Stai mescolando double precision
output di date_part()
con text '-'
. Questo non ha senso per PostgreSQL. Avresti bisogno di un cast esplicito per text
. Ma c'è un modo molto più semplice per fare tutto questo:
startdate:=date_part('year',to_timestamp(NEW.date))
||'-'||date_part('month',to_timestamp(NEW.date))
||'-'||date_part('day',to_timestamp(NEW.date));
Usa invece:
startdate := to_char(NEW.date, 'YYYY-MM-DD');
Neanche questo ha senso:
EXECUTE 'CREATE TABLE $1 (
CHECK (date >= DATE $2 AND date < DATE $3 )
) INHERITS (pings)' USING quote_ident(tablename),startdate,enddate;
Puoi fornire solo valori con USING
clausola. Leggi il manuale qui . Prova invece:
EXECUTE 'CREATE TABLE ' || quote_ident(tablename) || ' (
CHECK ("date" >= ''' || startdate || ''' AND
"date" < ''' || enddate || '''))
INHERITS (ping)';
O meglio ancora, usa format()
. Vedi sotto.
Inoltre, come @a_horse ha risposto :Devi mettere i tuoi valori di testo tra virgolette singole.
Simile qui:
EXECUTE 'INSERT INTO $1 VALUES (NEW.*)' USING quote_ident(tablename);
Invece:
EXECUTE 'INSERT INTO ' || quote_ident(tablename) || ' VALUES ($1.*)'
USING NEW;
Risposta correlata:
A parte:sebbene "data" sia consentita per il nome di una colonna in PostgreSQL, è un parola riservata in ogni standard SQL . Non nominare la tua colonna "data", porta a errori di sintassi confusi.
Completa demo funzionante
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();
-
Aggiornamento: Le versioni successive di Postgres hanno metodi più eleganti per verificare se esiste una tabella:
-
to_char()
può richiedere unadate
come$1
. Viene convertito intimestamp
automaticamente.
Il manuale sulle funzioni di data/ora . -
(Facoltativo)
SET
ilsearch_path
nell'ambito della tua funzione per evitare comportamenti scorretti con unsearch_path
modificato impostazione. -
Molteplici altre semplificazioni e miglioramenti. Confronta il codice.
Prove:
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);