Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: postgresql-cfbot/postgresql
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: cf/6021~1
Choose a base ref
...
head repository: postgresql-cfbot/postgresql
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: cf/6021
Choose a head ref
  • 5 commits
  • 14 files changed
  • 2 contributors

Commits on Oct 30, 2025

  1. Add ternary reloption type 

    There is a tendency for boolean reloptions in PostgreSQL code, one with "on"
and "off" values, to be replaced with options with "on", "off",
"[use_global_settings]" behaviour.

For `vacuum_index_cleanup" and gist's `buffering` reloption such behavior have
been implemented as enum-tyoe option.

For vacuum_truncate this behaviour have been umplemented by adding additional
`is_set` flag to `bytea` representation of the reloptions.

Both solutions looks like workaround hacks to implement option with three
available states.

This patch introduce "ternary" reloption type, that behave like bool option,
but also has an additional "unset" state. This state may be reachable only by
not setting or RESETting an option, like in `vacuum_truncate` option, or it
may have some text alias that allow user to explicitly set it, like "auto" in
`vacuum_index_cleanup` or gist's `buffering`

`vacuum_truncate`, `vacuum_index_cleanup` and gist's `buffering` reloptions are
reimplemented as ternary  reloptions without significant behaviour changes.
    @nataraj-hates-MS-for-stealing-github
    nataraj-hates-MS-for-stealing-github authored and Commitfest Bot committed Oct 30, 2025
    Configuration menu
    Copy the full SHA
    38bf1f3 View commit details
    Browse the repository at this point in the history

  2. Introduce ternary reloptions 

    Introduce ternary reloption as a replacement for current `vacuum_truncate`
implementation. Remove `vacuum_truncate_set` additional flag and using
`TERNARY_UNSET` value instead.
    @nataraj-hates-MS-for-stealing-github
    nataraj-hates-MS-for-stealing-github authored and Commitfest Bot committed Oct 30, 2025
    Configuration menu
    Copy the full SHA
    907c510 View commit details
    Browse the repository at this point in the history

  3. Add alias to be used as "unset" state. 

    Add `unset_alias` string parameter to ternary reloption definition. This will allow
user explicitly switch ternary option to "unset" state. Use this feature to
implement `vacuum_index_cleanup` and gist's `buffering` reloptions as ternary
reloptions
    @nataraj-hates-MS-for-stealing-github
    nataraj-hates-MS-for-stealing-github authored and Commitfest Bot committed Oct 30, 2025
    Configuration menu
    Copy the full SHA
    c23f0a5 View commit details
    Browse the repository at this point in the history

  4. Extra tests 

    Add more tests for ternary reloptions in dummy_index_am module
    @nataraj-hates-MS-for-stealing-github
    nataraj-hates-MS-for-stealing-github authored and Commitfest Bot committed Oct 30, 2025
    Configuration menu
    Copy the full SHA
    1d4e2a3 View commit details
    Browse the repository at this point in the history

  5. [CF 6021] ternary reloption type 

    This branch was automatically generated by a robot using patches from an
email thread registered at:

https://commitfest.postgresql.org/patch/6021

The branch will be overwritten each time a new patch version is posted to
the thread, and also periodically to check for bitrot caused by changes
on the master branch.

Patch(es): https://www.postgresql.org/message-id/4047390.3Lj2Plt8kZ@thinkpad-pgpro
Author(s): Nikolay Shaplov
    Commitfest Bot committed Oct 30, 2025
    Configuration menu
    Copy the full SHA
    282ba61 View commit details
    Browse the repository at this point in the history
Loading