| From: | Nikolay Samokhvalov <nik(at)postgres(dot)ai> |
|---|---|
| To: | Greg Sabino Mullane <htamfids(at)gmail(dot)com> |
| Cc: | pgsql-hackers mailing list <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_dump, pg_dumpall, pg_restore: Add --no-policies option |
| Date: | 2025-01-10 18:50:05 |
| Message-ID: | CAM527d-zc38oE-m3OW-t8O9ufq6+eUt73VaL6-9zUZa5u+abGQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Thank you Greg.
002_pg_dump.pl already deals with CREATE POLICY and ALTER TABLE .. ENABLE
ROW LEVEL SECURITY, so I just added "--no-policies" there, to have basic
coverage.
Nik
On Fri, Jan 10, 2025 at 9:44 AM Greg Sabino Mullane <htamfids(at)gmail(dot)com>
wrote:
> Looks good to me. Would ideally like to see some tests: should be easy
> enough to add to t/002_pg_dump.pl, but probably not worth it just for a
> simple flag like this? We don't test a lot of other flags, but on the other
> hand, that's what a test suite is supposed to do.
>
> Cheers,
> Greg
>
>
| Attachment | Content-Type | Size |
|---|---|---|
| 0002-pg_dump-pg_dumpall-pg_restore-Add-no-policies-option.patch | application/x-patch | 10.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2025-01-10 19:07:25 | Re: Reorder shutdown sequence, to flush pgstats later |
| Previous Message | Andres Freund | 2025-01-10 18:43:14 | Re: Reorder shutdown sequence, to flush pgstats later |