NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf313646

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 parentc6679e4 commitf313646Copy full SHA for f313646

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
`@@ -3666,6 +3666,8 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3666`	`3666`	`TableInfo *tbinfo = polinfo->poltable;`
`3667`	`3667`	`PQExpBuffer query;`
`3668`	`3668`	`PQExpBuffer delqry;`
	`3669`	`+PQExpBuffer polprefix;`
	`3670`	`+char *qtabname;`
`3669`	`3671`	`const char *cmd;`
`3670`	`3672`	`char *tag;`
`3671`	`3673`
`@@ -3723,6 +3725,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3723`	`3725`
`3724`	`3726`	`query = createPQExpBuffer();`
`3725`	`3727`	`delqry = createPQExpBuffer();`
	`3728`	`+polprefix = createPQExpBuffer();`
	`3729`	`+`
	`3730`	`+qtabname = pg_strdup(fmtId(tbinfo->dobj.name));`
`3726`	`3731`
`3727`	`3732`	`appendPQExpBuffer(query, "CREATE POLICY %s", fmtId(polinfo->polname));`
`3728`	`3733`
`@@ -3743,6 +3748,9 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3743`	`3748`	`appendPQExpBuffer(delqry, "DROP POLICY %s", fmtId(polinfo->polname));`
`3744`	`3749`	`appendPQExpBuffer(delqry, " ON %s;\n", fmtQualifiedDumpable(tbinfo));`
`3745`	`3750`
	`3751`	`+appendPQExpBuffer(polprefix, "POLICY %s ON",`
	`3752`	`+ fmtId(polinfo->polname));`
	`3753`	`+`
`3746`	`3754`	`tag = psprintf("%s %s", tbinfo->dobj.name, polinfo->dobj.name);`
`3747`	`3755`
`3748`	`3756`	`if (polinfo->dobj.dump & DUMP_COMPONENT_POLICY)`
`@@ -3755,9 +3763,16 @@ dumpPolicy(Archive fout, PolicyInfo polinfo)`
`3755`	`3763`	`.createStmt = query->data,`
`3756`	`3764`	`.dropStmt = delqry->data));`
`3757`	`3765`
	`3766`	`+if (polinfo->dobj.dump & DUMP_COMPONENT_COMMENT)`
	`3767`	`+dumpComment(fout, polprefix->data, qtabname,`
	`3768`	`+tbinfo->dobj.namespace->dobj.name, tbinfo->rolname,`
	`3769`	`+polinfo->dobj.catId, 0, polinfo->dobj.dumpId);`
	`3770`	`+`
`3758`	`3771`	`free(tag);`
`3759`	`3772`	`destroyPQExpBuffer(query);`
`3760`	`3773`	`destroyPQExpBuffer(delqry);`
	`3774`	`+destroyPQExpBuffer(polprefix);`
	`3775`	`+free(qtabname);`
`3761`	`3776`	`}`
`3762`	`3777`
`3763`	`3778`	`/*`

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!

Commitf313646

File tree

1 file changed

1 file changed

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

0 commit comments