pgsql: Allow Boolean reloptions to have ternary values
| От | Álvaro Herrera |
|---|---|
| Тема | pgsql: Allow Boolean reloptions to have ternary values |
| Дата | |
| Msg-id | E1vidYa-001qhJ-11@gemulon.postgresql.org обсуждение исходный текст |
| Список | pgsql-committers |
Allow Boolean reloptions to have ternary values From the user's point of view these are just Boolean values; from the implementation side we can now distinguish an option that hasn't been set. Reimplement the vacuum_truncate reloption using this type. This could also be used for reloptions vacuum_index_cleanup and buffering, but those additionally need a per-option "alias" for the state where the variable is unset (currently the value "auto"). Author: Nikolay Shaplov <dhyan@nataraj.su> Reviewed-by: Timur Magomedov <t.magomedov@postgrespro.ru> Reviewed-by: Nathan Bossart <nathandbossart@gmail.com> Discussion: https://postgr.es/m/3474141.usfYGdeWWP@thinkpad-pgpro Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/4d6a66f675815a5d40a650d4dcfb5ddb89c6ad2f Modified Files -------------- src/backend/access/common/reloptions.c | 138 +++++++++++++++++---- src/backend/commands/vacuum.c | 4 +- src/include/access/reloptions.h | 26 ++-- src/include/postgres.h | 14 +++ src/include/utils/rel.h | 3 +- src/test/modules/dummy_index_am/README | 2 +- src/test/modules/dummy_index_am/dummy_index_am.c | 55 +++++--- .../modules/dummy_index_am/expected/reloptions.out | 24 +++- src/test/modules/dummy_index_am/sql/reloptions.sql | 10 ++ src/test/regress/expected/reloptions.out | 18 +++ src/test/regress/sql/reloptions.sql | 11 ++ src/tools/pgindent/typedefs.list | 2 + 12 files changed, 242 insertions(+), 65 deletions(-)
В списке pgsql-committers по дате отправления: