NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf9d7bb6

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 parent1beab07 commitf9d7bb6Copy full SHA for f9d7bb6

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
`@@ -3333,6 +3333,8 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3333`	`3333`	`TableInfo *tbinfo = polinfo->poltable;`
`3334`	`3334`	`PQExpBuffer query;`
`3335`	`3335`	`PQExpBuffer delqry;`
	`3336`	`+PQExpBuffer polprefix;`
	`3337`	`+char *qtabname;`
`3336`	`3338`	`const char *cmd;`
`3337`	`3339`	`char *tag;`
`3338`	`3340`
`@@ -3390,6 +3392,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3390`	`3392`
`3391`	`3393`	`query = createPQExpBuffer();`
`3392`	`3394`	`delqry = createPQExpBuffer();`
	`3395`	`+polprefix = createPQExpBuffer();`
	`3396`	`+`
	`3397`	`+qtabname = pg_strdup(fmtId(tbinfo->dobj.name));`
`3393`	`3398`
`3394`	`3399`	`appendPQExpBuffer(query, "CREATE POLICY %s", fmtId(polinfo->polname));`
`3395`	`3400`
`@@ -3410,6 +3415,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3410`	`3415`	`appendPQExpBuffer(delqry, "DROP POLICY %s", fmtId(polinfo->polname));`
`3411`	`3416`	`appendPQExpBuffer(delqry, " ON %s;\n", fmtQualifiedDumpable(tbinfo));`
`3412`	`3417`
	`3418`	`+appendPQExpBuffer(polprefix, "POLICY %s ON",`
	`3419`	`+ fmtId(polinfo->polname));`
	`3420`	`+`
`3413`	`3421`	`tag = psprintf("%s %s", tbinfo->dobj.name, polinfo->dobj.name);`
`3414`	`3422`
`3415`	`3423`	`if (polinfo->dobj.dump & DUMP_COMPONENT_POLICY)`
`@@ -3423,9 +3431,16 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3423`	`3431`	`NULL, 0,`
`3424`	`3432`	`NULL, NULL);`
`3425`	`3433`
	`3434`	`+if (polinfo->dobj.dump & DUMP_COMPONENT_COMMENT)`
	`3435`	`+dumpComment(fout, polprefix->data, qtabname,`
	`3436`	`+tbinfo->dobj.namespace->dobj.name, tbinfo->rolname,`
	`3437`	`+polinfo->dobj.catId, 0, polinfo->dobj.dumpId);`
	`3438`	`+`
`3426`	`3439`	`free(tag);`
`3427`	`3440`	`destroyPQExpBuffer(query);`
`3428`	`3441`	`destroyPQExpBuffer(delqry);`
	`3442`	`+destroyPQExpBuffer(polprefix);`
	`3443`	`+free(qtabname);`
`3429`	`3444`	`}`
`3430`	`3445`
`3431`	`3446`	`/*`

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!

Commitf9d7bb6

File tree

1 file changed

1 file changed

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

0 commit comments