]> granicus.if.org Git - postgresql/commit
Refactor GUC set_config_option function:
authorBruce Momjian <bruce@momjian.us>
Fri, 11 Aug 2006 20:08:28 +0000 (20:08 +0000)
committerBruce Momjian <bruce@momjian.us>
Fri, 11 Aug 2006 20:08:28 +0000 (20:08 +0000)
commitf91ddb768b9d29bec1b7c2e039a5d2e55a25dd2c
tree6f87dfe05f6f67fe2772431f46fc1f19399a83bb
parentc07fbcf57701a26dfafd44ebe3edc4070fa6c6d0
Refactor GUC set_config_option function:

The main reason for refactoring was that set_config_option() was too
overloaded function and its behavior did not consistent. Old version of
set_config_function hides some messages. For example if you type:

tcp_port = 5432.1

then old implementation ignore this error without any message to log
file in the signal context (configuration reload). Main problem was that
semantic analysis of postgresql.conf is not perform in the
ProcessConfigFile function, but in the set_config_options *after*
context check. This skipped check for variables with PG_POSTMASTER
context. There was request from Joachim Wieland to add more messages
about ignored changes in the config file as well.

Zdenek Kotala
src/backend/utils/misc/guc-file.l
src/backend/utils/misc/guc.c
src/include/utils/guc.h