Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES
Дата
Msg-id YWfh9O/36r+GlgNB@paquier.xyz
обсуждение исходный текст
Ответ на Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES  (Masahiko Sawada <sawada.mshk@gmail.com>)
Ответы Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES  ("Bossart, Nathan" <bossartn@amazon.com>)
Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES  ("Bossart, Nathan" <bossartn@amazon.com>)
Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES  (Masahiko Sawada <sawada.mshk@gmail.com>)
Список pgsql-bugs
On Thu, Oct 14, 2021 at 02:22:21PM +0900, Masahiko Sawada wrote:
> Agreed. Please find an attached new patch.

I have not dived into the details of the patch yet, but I can see the
following diffs in some of the dumps dropped by the new test added
between HEAD and the patch:
1) For DEFAULT PRIVILEGES FOR FUNCTIONS:
-ALTER DEFAULT PRIVILEGES FOR ROLE regress_dump_test_role IN SCHEMA
     dump_test REVOKE ALL ON FUNCTIONS  FROM PUBLIC;
+ALTER DEFAULT PRIVILEGES FOR ROLE regress_dump_test_role IN SCHEMA
     dump_test GRANT ALL ON FUNCTIONS  TO regress_dump_test_role;
2) For DEFAULT PRIVILEGES FOR TABLES:
-ALTER DEFAULT PRIVILEGES FOR ROLE regress_dump_test_role IN SCHEMA
     dump_test REVOKE ALL ON TABLES  FROM regress_dump_test_role;
 ALTER DEFAULT PRIVILEGES FOR ROLE regress_dump_test_role IN SCHEMA
     dump_test GRANT SELECT ON TABLES  TO regress_dump_test_role;

So the patch removes a REVOKE ALL ON TABLES on
regress_dump_test_role after the addition of only the GRANT EXECUTE ON
FUNCTIONS.  That seems off.  Am I missing something?
--
Michael

Вложения

В списке pgsql-bugs по дате отправления:

Предыдущее
От: Masahiko Sawada
Дата:
Сообщение: Re: Inconsistent behavior of pg_dump/pg_restore on DEFAULT PRIVILEGES
Следующее
От: PG Bug reporting form
Дата:
Сообщение: BUG #17230: The postmaster has commanded this server process to roll back the current transaction and exit, beca