diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-10-03 16:40:06 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-10-03 16:40:26 -0400 |
| commit | 993d94c59038fb80068a2b7780a1062dcaa2bb4f (patch) | |
| tree | 9ccf8e08145cf9619feb0e225d8f018cb1e9291f /src/backend/executor/execProcnode.c | |
| parent | 190765a059cc5e8226cdb18b9715741b32b12255 (diff) | |
| download | postgresql-993d94c59038fb80068a2b7780a1062dcaa2bb4f.tar.gz postgresql-993d94c59038fb80068a2b7780a1062dcaa2bb4f.zip | |
Show a sensible value in pg_settings.unit for GUC_UNIT_XSEGS variables.
Commit 88e982302 invented GUC_UNIT_XSEGS for min_wal_size and max_wal_size,
but neglected to make it display sensibly in pg_settings.unit (by adding a
case to the switch in GetConfigOptionByNum). Fix that, and adjust said
switch to throw a run-time error the next time somebody forgets.
In passing, avoid using a static buffer for the output string --- the rest
of this function pstrdup's from a local buffer, and I see no very good
reason why the units code should do it differently and less safely.
Per report from Otar Shavadze. Back-patch to 9.5 where the new unit type
was added.
Report: <CAG-jOyA=iNFhN+yB4vfvqh688B7Tr5SArbYcFUAjZi=0Exp-Lg@mail.gmail.com>
Diffstat (limited to 'src/backend/executor/execProcnode.c')
0 files changed, 0 insertions, 0 deletions