Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitcb11f4d

Browse files
committed
Fix processing of PGC_BACKEND GUC parameters on Windows.
EXEC_BACKEND builds (i.e., Windows) failed to absorb values of PGC_BACKENDparameters if they'd been changed post-startup via the config file. Thisfor example prevented log_connections from working if it were turned onpost-startup. The mechanism for handling this case has always been a bitof a kluge, and it wasn't revisited when we implemented EXEC_BACKEND.While in a normal forking environment new backends will inherit thepostmaster's value of such settings, EXEC_BACKEND backends have to readthe settings from the CONFIG_EXEC_PARAMS file, and they were mistakenlyrejecting them. So this case has always been broken in the Windows port;so back-patch to all supported branches.Amit Kapila
1 parentaf7738f commitcb11f4d

File tree

1 file changed

+20
-0
lines changed
  • src/backend/utils/misc

1 file changed

+20
-0
lines changed

‎src/backend/utils/misc/guc.c‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5191,9 +5191,23 @@ set_config_option(const char *name, const char *value,
51915191
* ignore it in existing backends.This is a tad klugy, but
51925192
* necessary because we don't re-read the config file during
51935193
* backend start.
5194+
*
5195+
* In EXEC_BACKEND builds, this works differently: we load all
5196+
* nondefault settings from the CONFIG_EXEC_PARAMS file during
5197+
* backend start. In that case we must accept PGC_SIGHUP
5198+
* settings, so as to have the same value as if we'd forked
5199+
* from the postmaster. We detect this situation by checking
5200+
* IsInitProcessingMode, which is a bit ugly, but it doesn't
5201+
* seem worth passing down an explicit flag saying we're doing
5202+
* read_nondefault_variables().
51945203
*/
5204+
#ifdefEXEC_BACKEND
5205+
if (IsUnderPostmaster&& !IsInitProcessingMode())
5206+
return true;
5207+
#else
51955208
if (IsUnderPostmaster)
51965209
return true;
5210+
#endif
51975211
}
51985212
elseif (context!=PGC_POSTMASTER&&context!=PGC_BACKEND&&
51995213
source!=PGC_S_CLIENT)
@@ -7514,6 +7528,12 @@ read_nondefault_variables(void)
75147528
intvarsourceline;
75157529
GucSourcevarsource;
75167530

7531+
/*
7532+
* Assert that PGC_BACKEND case in set_config_option() will do the right
7533+
* thing.
7534+
*/
7535+
Assert(IsInitProcessingMode());
7536+
75177537
/*
75187538
* Open file
75197539
*/

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp