2

我有一个表address_all，它被几个地址表继承。address_history从父表继承history_all并保留当前地址信息。我正在创建继承address_all表并将信息从新表复制address_history到新表的新表。

我的存储过程如下所示。当我调用它时，我遇到了一些错误。为了更好地解释错误，我使用了行号。

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;

当我运行时：

select somefunc()

我收到此错误：

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
4

3 回答 3

6

试试这个大大简化的形式：

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;

要点：

  • 您可以在声明时在 plpgsql 中分配变量。简化代码。

  • 用于to_char()格式化您的日期。简单得多。

  • now()并CURRENT_TIMESTAMP做同样的事情。

  • 如果您想要当前时间戳，请不要引用'now()'，使用（不带引号）。now()

  • 使用该USING子句 withEXECUTE，因此您不必转换timestamptotext和 back - 可能会像您一样遇到引用问题。更快、更简单、更安全。

  • 在LANGUAGE plpgsql,plpgsql是一个关键字，不应被引用。

  • 您可能想检查表是否已经存在CREATE TABLE IF NOT EXISTS，自 PostgreSQL 9.1 起可用。

于 2012-08-07T17:05:13.193 回答
2

显然您需要引用backupdays，否则它不会被视为解析时间戳的字符串。

于 2012-08-07T16:27:53.683 回答
1

您正在使用字符串操作构建 SQL，因此您必须像使用任何其他语言一样正确引用所有内容。您需要了解一些功能：

  • quote_ident: 引用一个标识符，例如表名。
  • quote_literal: 引用一个字符串以用作字符串文字。
  • quote_nullable: 一样，quote_literal但也可以正确处理 NULL。

这样的事情会更好地为您服务：

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

在您的情况下，这些quote_ident电话不是必需的，但它们是一个好习惯。

于 2012-08-07T17:07:32.927 回答