NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitbd2cd13

committed

Teach pg_dump to dump comments on RLS policy objects.

This was unaccountably omitted in the original RLS patch.The SQL syntax is basically the same as for comments on triggers,so crib code from dumpTrigger().Per report from Marc Munro. Back-patch to all supported branches.Discussion:https://postgr.es/m/1581889298.18009.15.camel@bloodnok.com

1 parent12bea22 commitbd2cd13Copy full SHA for bd2cd13

File tree

1 file changed

+15

-0

lines changed

src/bin/pg_dump
- pg_dump.c

1 file changed

+15

-0

lines changed

`‎src/bin/pg_dump/pg_dump.c`

Lines changed: 15 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -3618,6 +3618,8 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3618`	`3618`	`TableInfo *tbinfo = polinfo->poltable;`
`3619`	`3619`	`PQExpBuffer query;`
`3620`	`3620`	`PQExpBuffer delqry;`
	`3621`	`+PQExpBuffer polprefix;`
	`3622`	`+char *qtabname;`
`3621`	`3623`	`const char *cmd;`
`3622`	`3624`	`char *tag;`
`3623`	`3625`
`@@ -3675,6 +3677,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3675`	`3677`
`3676`	`3678`	`query = createPQExpBuffer();`
`3677`	`3679`	`delqry = createPQExpBuffer();`
	`3680`	`+polprefix = createPQExpBuffer();`
	`3681`	`+`
	`3682`	`+qtabname = pg_strdup(fmtId(tbinfo->dobj.name));`
`3678`	`3683`
`3679`	`3684`	`appendPQExpBuffer(query, "CREATE POLICY %s", fmtId(polinfo->polname));`
`3680`	`3685`
`@@ -3695,6 +3700,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3695`	`3700`	`appendPQExpBuffer(delqry, "DROP POLICY %s", fmtId(polinfo->polname));`
`3696`	`3701`	`appendPQExpBuffer(delqry, " ON %s;\n", fmtQualifiedDumpable(tbinfo));`
`3697`	`3702`
	`3703`	`+appendPQExpBuffer(polprefix, "POLICY %s ON",`
	`3704`	`+ fmtId(polinfo->polname));`
	`3705`	`+`
`3698`	`3706`	`tag = psprintf("%s %s", tbinfo->dobj.name, polinfo->dobj.name);`
`3699`	`3707`
`3700`	`3708`	`if (polinfo->dobj.dump & DUMP_COMPONENT_POLICY)`
`@@ -3708,9 +3716,16 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3708`	`3716`	`NULL, 0,`
`3709`	`3717`	`NULL, NULL);`
`3710`	`3718`
	`3719`	`+if (polinfo->dobj.dump & DUMP_COMPONENT_COMMENT)`
	`3720`	`+dumpComment(fout, polprefix->data, qtabname,`
	`3721`	`+tbinfo->dobj.namespace->dobj.name, tbinfo->rolname,`
	`3722`	`+polinfo->dobj.catId, 0, polinfo->dobj.dumpId);`
	`3723`	`+`
`3711`	`3724`	`free(tag);`
`3712`	`3725`	`destroyPQExpBuffer(query);`
`3713`	`3726`	`destroyPQExpBuffer(delqry);`
	`3727`	`+destroyPQExpBuffer(polprefix);`
	`3728`	`+free(qtabname);`
`3714`	`3729`	`}`
`3715`	`3730`
`3716`	`3731`	`/*`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitbd2cd13

File tree

1 file changed

1 file changed

`‎src/bin/pg_dump/pg_dump.c`

0 commit comments