Hi,
When I played with regression tests for pg_restore, I tested -T filtering
triggers too. I had problems with restoring triggers. I found that the name
for trigger uses the pattern "tablename triggername" (not just (and
correct) triggername).
I propose to generate tag just like trigger name
proposed patch attached
regards
Pavel
diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index da427f4d4a..8ffc6024ea 100644
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -17148,7 +17148,7 @@ dumpTrigger(Archive *fout, const TriggerInfo *tginfo)
appendPQExpBuffer(trigprefix, "TRIGGER %s ON",
fmtId(tginfo->dobj.name));
- tag = psprintf("%s %s", tbinfo->dobj.name, tginfo->dobj.name);
+ tag = psprintf("%s", tginfo->dobj.name);
if (tginfo->dobj.dump & DUMP_COMPONENT_DEFINITION)
ArchiveEntry(fout, tginfo->dobj.catId, tginfo->dobj.dumpId,