postgreSQLのトリガー関数、ストアドプロシージャでエラー発生

Question

ただいまpostgreSQLのトリガー勉強中です・・・！がエラー発生しており原因と対応が不明なので教えてほしいです

やったこと

以下のストアドプロシージャを作りました。ユーザデータを新規作成後、自動でtest_flgにyをいれるようにしています

ストアドプロシージャ作成

CREATE OR REPLACE FUNCTION update_user() RETURNS TRIGGER AS $$
    BEGIN
        UPDATE User SET test_flg='Y' WHERE test_flg='N' and No like 'No-M%';
    END;
$$ LANGUAGE plpgsql;

トリガー関数として登録

CREATE TRIGGER check_update
    AFTER INSERT ON User
    FOR EACH ROW
    EXECUTE PROCEDURE update_user();

発生しているエラー

DBExecuteError:ERROR: control reached end of trigger procedure without RETURN - Where: PL/pgSQL function 

返り値は今回とくに必要ないのですが、RETURNのエラーがでています。
こちらどう対応すればいいのでしょうか・・・

Answer 1

CREATE OR REPLACE FUNCTION update_user() RETURNS TRIGGER AS $$

FUNCTION では RETURN が必要では？

だから、

trigger procedure without RETURN - Where: PL/pgSQL function

って怒られています。

EXECUTE PROCEDURE update_user();

FUNCTION のはずが、PROCEDURE で呼んでいる。

test_flg='N' and No like 'No-M%' だったら、test_flg に 'Y' を設定するだけのコードを BEFORE INSERT トリガーに設定するだけで良いのでは？

Comments

[sazi]

>FUNCTION のはずが、PROCEDURE で呼んでいる。
postgresにはprocedureは無いので、互換性の為にどちらでも良いようになっているんだと思います。

[Orlofsky]

それは知りませんでした。ご指摘ありがとうございます。

[kakeiteiya]

回答ありがとうございます！！

Answer 2

そもそもトリガー用のFunction(RETURNS TRIGGERが必須）なのですから、必ずreturnは必要です。
※トリガーとは関係なく、戻り値の無いFunction(returns void)の場合はreturnは不要ですが。

返り値は今回とくに必要ない

return Null;

を追加して下さい。

Comments

[kakeiteiya]

ありがとうございます！！！！！！！！！！！！！！！！！！
できました！！！！！！！！

[Orlofsky]

INSERT直後のテーブルをUPDATEするのってパフォーマンスが悪いのでは？
INSERT直前に IF で条件を指定して代入すれば済むだけでは？
https://qiita.com/grassfield/items/60fad9c1d4a8d205bff3

[sazi]

まあ、勉強中という事なので、そのためにトリガーを試しているだけだと思いますよ。

[Orlofsky]

半端な指導をするなんてsaziさんらしくないと思いまして。

[sazi]

トリガーの勉強をしているんだからという事で、トリガーを使ってみる為にエラーを解消させる方法を回答しました。
処理内容については取り敢えずで記述している位にしか見てませんでしたね。

[kakeiteiya]

>INSERT直後のテーブルをUPDATEするのってパフォーマンスが悪いのでは？
>INSERT直前に IF で条件を指定して代入すれば済むだけでは？
ほんとですね・・・????

これ見直します！ありがとうございます！

[kakeiteiya]

＞トリガーを使ってみる為に
ありがとうございます！
まずは動かすことが目的でした！