«выполнить создание индекса» в plpgsql не запускается

Question

У меня возникают проблемы с выполнением «выполнить создание индекса» внутри функции plgpsql (postgres 9.4). Например:

create or replace function foo() returns void language plpgsql as $$ 
begin 
    perform 'create unique index patients_row_id_key on patients(row_id)'; 
end; $$; 

Кажется работать нормально:

select foo(); 

Однако индекс не создается. Любой диагноз и обходной путь? Я пробовал:

alter function foo() VOLATILE; 

и все еще не повезло.

Answer 1

PERFORM заявление в PLPGSQL, используемое для выполнения запросов, которые не возвращают результат или какой результат не полезен. Технически PERFORM ... внутри блока PLPGSQL равен SELECT ... в простом SQL. Поэтому в вашем примере вы пытаетесь выполнить что-то вроде

select 'create unique index patients_row_id_key on patients(row_id)'; 

и просто проигнорировать результат.

Подробнее: Executing a Command With No Result

Вы не должны обернуть заявления DDL внутри PLPGSQL и может использовать его как:

create or replace function foo() returns void language plpgsql as $$ 
begin 
    create unique index patients_row_id_key on patients(row_id); 
end; $$; 

Или, если вы хотите построить его во время выполнения, то используйте EXECUTE заявление: Executing Dynamic Commands например:

create or replace function foo(p_tablename text) returns void language plpgsql as $$ 
begin 
    execute 'create unique index ' || p_tablename || '_row_id_key on ' || p_tablename || '(row_id)'; 
end; $$; 

Answer 2

В качестве дополнения к пункту выполнения, обратите внимание на два важных момента. (! Опасная)

1. Вы делаете интерполяцию строки с SQL запросами, и
2. Вы должны использовать quote_ident, не quote_literal

Если вы используете функцию Abelisto по выше, и вызвать его:

SELECT foo('test_idx on test; drop table foo; --'); 

SQL-инъекция в хранимой процедуре. Хуже, если это определение безопасности. Исправленная версия будет:

create or replace function foo(p_tablename text) returns void language plpgsql as $$ 
begin 
    execute 'create unique index ' || quote_ident(p_tablename || '_row_id_key') || ' on ' || quote_ident(p_tablename) || '(row_id)'; 
end; $$; 

Answer 3

What @Abelisto wrote о PERFORM.
И what @Chris added о внедрении SQL.

Плюс, я предлагаю использовать format() для чего угодно, кроме самых тривиальных строк запросов, чтобы упростить вашу жизнь с помощью динамического SQL. And the manual does, too:

Очиститель подход заключается в использовании %I спецификации format() «ы для таблиц и столбцов.

CREATE OR REPLACE FUNCTION foo(_tbl text) 
    RETURNS void AS 
$func$ 
BEGIN 
    EXECUTE format('CREATE UNIQUE INDEX %I ON %I(row_id)', _tbl || _row_id_key', _tbl); 
END 
$func$ LANGUAGE plpgsql; 

regclass параметра является удобной альтернативой для передачи имен таблиц, но конкатенаций новых идентификаторов может быть сложнее - как это недавно соответствующим случай идет, чтобы показать:

• PL/pgSQL regclass quoting of table named like keyword