PostgreSQL插入后触发器写入日志表报错，位置171求助

我帮你排查下这段代码里的几个关键问题，这些应该就是导致你在位置171报错的根源：

1. 参数类型不兼容 #

你的insert_log函数最后一个参数定义成了VARCHAR，但触发器里传递的row_to_json(NEW)是JSON类型，类型不匹配直接会触发语法/类型错误。

2. 触发器参数引用错误 #

• 触发器里要获取新插入行的主键，得写NEW.id，直接写idPostgreSQL找不到这个变量；
• 你用双引号包了"clientes"，但在PostgreSQL里双引号是用来引用标识符（比如表名、列名）的，字符串常量必须用单引号包裹，不然数据库会把它当成一个列名去查找，自然会报错。

3. 函数返回值处理有问题 #

函数声明要返回INT，但你当前的INSERT ... RETURNING id没有把结果返回出去，得把这个返回值存到变量里再用RETURN返回，或者直接用RETURN (INSERT ...)的写法（不过用变量更清晰）。

4. 关键字字段的引用注意 #

日志表的primary是PostgreSQL的保留关键字，虽然建表时用双引号定义了，但在函数里插入数据时最好保持用双引号包裹这个字段，避免歧义。

修正后的完整代码 #

先修正日志函数 #

CREATE OR REPLACE FUNCTION insert_log (p_action INT, p_table VARCHAR, p_primary BIGINT, p_log JSON) 
RETURNS INT AS $$
DECLARE
    v_log_id BIGINT;
BEGIN
    INSERT INTO public.logs ("action", "table", "primary", "log") 
    VALUES (p_action, p_table, p_primary, p_log) 
    RETURNING id INTO v_log_id;
    
    RETURN v_log_id;
END $$ LANGUAGE plpgsql;

再创建触发器 #

CREATE TRIGGER trigger_clientes_insert 
AFTER INSERT ON xxx.legacy_clientes 
FOR EACH ROW 
EXECUTE FUNCTION insert_log (1, 'clientes', NEW.id, row_to_json(NEW));

额外优化建议 #

• 给函数参数加个前缀（比如p_），能有效避免和表字段名冲突，代码可读性也更高；
• 如果后续要处理更新、删除操作，可以扩展这个函数（比如加个操作类型判断），或者单独建对应的日志函数，比如update_log（需要同时传递OLD和NEW数据）、delete_log；
• 可以给日志表的几个字段设置默认值，减少插入时的代码量：

  ALTER TABLE public.logs 
  ALTER COLUMN processed SET DEFAULT FALSE,
  ALTER COLUMN created_at SET DEFAULT NOW(),
  ALTER COLUMN updated_at SET DEFAULT NOW();
  

内容的提问来源于stack exchange，提问作者Danilo Kobold