État SQL: 42601 erreur de syntaxe sur ou près de “11”

J'ai une table address_all et elle est héritée par plusieurs tables d'adresse. address_history hérite de table parent history_all et tient à jour les informations d'adresse. Je suis entrain de créer une nouvelle table qui hérite address_all table et des copies de l'information à partir de address_history à la nouvelle table.

Ma procédure stockée est comme ci-dessous. J'ai des erreurs quand je l'appelle. Pour mieux expliquer l'erreur, je suis en utilisant le numéro de ligne.

1  CREATE OR REPLACE FUNCTION somefunc()
2  RETURNS void AS
3  $BODY$
4  DECLARE
5   year_id INTEGER;
6   month_id INTEGER;
7   week_id INTEGER;
8   addresstablename text; 
9   backupdays text;
10 BEGIN
11  week_id := EXTRACT(DAY FROM TIMESTAMP 'now()');
12  month_id := EXTRACT(MONTH FROM TIMESTAMP 'now()');
13  year_id := EXTRACT(YEAR FROM TIMESTAMP 'now()');
14  addresstablename := 'address_history_' || week_id || '_' || month_id || '_' || year_id;
15  backupdays:= date_trunc('hour',CURRENT_TIMESTAMP - interval '7 days');
16  EXECUTE 'create table ' || addresstablename || '() INHERITS (address_all)';
17  EXECUTE 'insert into ' || addresstablename || ' select * from address_history where address_timestamp >= ' || backupdays || ''; --AS timestamp without time zone);  
18 END;
19 $BODY$
20 LANGUAGE 'plpgsql' VOLATILE;

Quand je le lance:

select somefunc()

J'obtiens cette erreur:

ERROR:  syntax error at or near "12"
LINE 1: ...story where address_timestamp >= 2012-07-31 12:00:00-0...
                                                         ^
QUERY:  insert into address_history_7_8_2012 select * from address_history where address_timestamp >= 2012-07-31 12:00:00-04
CONTEXT:  PL/pgSQL function "somefunc" line 14 at EXECUTE statement

 ********** Error **********

ERROR: syntax error at or near "12"
SQL state: 42601
Context: PL/pgSQL function "somefunc" line 14 at EXECUTE statement
InformationsquelleAutor PRP | 2012-08-07
  1. 6

    Essayer ce en grande partie une forme simplifiée:

    CREATE OR REPLACE FUNCTION somefunc()
  RETURNS void AS
$func$
DECLARE
 addresstablename text := 'address_history_' || to_char(now(), 'FMDD_MM_YYYY');

BEGIN
 EXECUTE 
 'CREATE TABLE ' || addresstablename || '() INHERITS (address_all)';

 EXECUTE
 'INSERT INTO ' || addresstablename || '
  SELECT *
  FROM   address_history
  WHERE  address_timestamp >= $1'
 USING date_trunc('hour', now() - interval '7 days');

END
$func$ LANGUAGE plpgsql;

    Les points principaux:

    • Vous pouvez attribuer des variables en plpgsql au moment de la déclaration. Simplifie le code.

    • Utilisation to_char() au format de votre date. Beaucoup plus simple.

    • now() et CURRENT_TIMESTAMP faire de même.

    • Ne pas citer 'now()', utilisez now() (sans les guillemets) si vous voulez le timestamp actuel.

    • Utiliser le à l'AIDE de clause EXECUTE, de sorte que vous n'avez pas à convertir les timestamp à text et à l'arrière - peut-être en cours d'exécution dans citant questions comme vous l'avez fait. Plus rapide, plus simple, plus sûr.

    • Dans LANGUAGE plpgsql, plpgsql est un mot-clé et ne doit pas être cité.

    • Vous voudrez peut-être vérifier si la table existe déjà avec CREATE TABLE if not EXISTS, disponible depuis PostgreSQL 9.1.

    • Merci de fournir plus d'aider, à ma requête. Je suis nouveau sur postgres.
    InformationsquelleAutor Erwin Brandstetter
  2. 2

    Apparemment, vous avez besoin de citer backupdays, ou il n'est pas vu comme une chaîne de caractères à partir d'où pour analyser un horodatage.

    InformationsquelleAutor LSerni
  3. 1

    Vous êtes la construction de SQL à l'aide de la manipulation de la chaîne de sorte que vous avez de bien citer tout comme dans toute autre langue. Il y a un quelques fonctions que vous voulez savoir sur:

    • quote_ident: citation d'un identificateur, tel qu'un nom de table.
    • quote_literal: citation d'une chaîne pour l'utiliser comme une chaîne littérale.
    • quote_nullable: comme quote_literal mais gère correctement les valeurs Null en tant que bien.

    Quelque chose de ce genre de serveur, vous mieux:

    EXECUTE 'create table ' || quote_ident(addresstablename) || ...
EXECUTE 'insert into '  || quote_ident(addresstablename) || ... || quote_literal(backupdays) ...

    La quote_ident appels ne sont pas nécessaires dans votre cas, mais ils sont une bonne habitude.

    • quote_literal() et quote_ident() sont de bons conseils, en général. Dans ce cas, vous n'en avez pas besoin, je dirais. Le nom de la table addresstablename est générée à l'intérieur de la fonction, et ne prend pas la saisie de l'utilisateur, de sorte qu'il peut être utilisé comme il est. La variable backupdays peut être appliquée avec le USING clause, qui permet d'économiser de moulage et de citer tout à fait.
    • Merci pour le partage de quote_ident, quote_literal et quote_nullable. Je vais sûrement l'utiliser dans mes requêtes. Merci beaucoup.
    • D'où la "La quote_ident appels ne sont pas nécessaires dans votre cas, mais ils sont une bonne habitude." Je dirais que quote_ident doit être la valeur par défaut chose que vos doigts, puis supprimez-la si vous êtes certain que vous n'en avez pas besoin.
    • Je ne peux pas en désaccord avec ça. Ne vous méprenez pas, je upvoted votre réponse - c'est la version deluxe de l'lserni du simple commentaire.
    • Pas de soucis, je suis juste de mettre sur ma "base de données de type" chapeau et d'essayer de s'assurer que tout est bien défini et sans ambiguïté 🙂
    InformationsquelleAutor mu is too short