Thread: Remove confusing commented-defaullts from postgresql.conf

Remove confusing commented-defaullts from postgresql.conf

From
"Greg Sabino Mullane"
Date:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
NotDashEscaped: You need GnuPG to verify this message


I know we never reached a full consensus on hackers about the
full solution to the postgresql.conf file, but I think everyone
agreed that having commented-out "default" values was a bad thing,
which caused confusion among even seasoned pg people. Thus, the
patch below which simply removes the starting '#' from all settings.
Forgive the lack of -c to the diff: I figured saving space was
more important as the context is not needed in this case.

--
Greg Sabino Mullane greg@turnstep.com
PGP Key: 0x14964AC8 200407122051




Index: postgresql.conf.sample
===================================================================
RCS file: /projects/cvsroot/pgsql-server/src/backend/utils/misc/postgresql.conf.sample,v
retrieving revision 1.115
diff -r1.115 postgresql.conf.sample
5c5
< # This file consists of lines of the form:
---
> # This file consists of lines of the form: GREG
11,12c11
< # allowed values can be found in the PostgreSQL documentation. The
< # commented-out settings shown in this file represent the default values.
---
> # allowed values can be found in the PostgreSQL documentation.
28,31c27,30
< # pgdata = '/usr/local/pgsql/data'        # use data in another directory
< # hba_conf = '/etc/pgsql/pg_hba.conf'         # use hba info in another directory
< # ident_conf = '/etc/pgsql/pg_ident.conf'    # use ident info in another directory
< # external_pidfile= '/var/run/postgresql.pid'    # write an extra pid file
---
> pgdata = '/usr/local/pgsql/data'        # use data in another directory
> hba_conf = '/etc/pgsql/pg_hba.conf'         # use hba info in another directory
> ident_conf = '/etc/pgsql/pg_ident.conf'    # use ident info in another directory
> external_pidfile= '/var/run/postgresql.pid'    # write an extra pid file
40c39
< #listen_addresses = 'localhost'    # what IP interface(s) to listen on;
---
> listen_addresses = 'localhost'    # what IP interface(s) to listen on;
42,43c41,42
< #port = 5432
< #max_connections = 100
---
> port = 5432
> max_connections = 100
47,51c46,50
< #superuser_reserved_connections = 2
< #unix_socket_directory = ''
< #unix_socket_group = ''
< #unix_socket_permissions = 0777    # octal
< #rendezvous_name = ''        # defaults to the computer name
---
> superuser_reserved_connections = 2
> unix_socket_directory = ''
> unix_socket_group = ''
> unix_socket_permissions = 0777    # octal
> rendezvous_name = ''        # defaults to the computer name
55,59c54,58
< #authentication_timeout = 60    # 1-600, in seconds
< #ssl = false
< #password_encryption = true
< #krb_server_keyfile = ''
< #db_user_namespace = false
---
> authentication_timeout = 60    # 1-600, in seconds
> ssl = false
> password_encryption = true
> krb_server_keyfile = ''
> db_user_namespace = false
68,77c67,76
< #shared_buffers = 1000        # min 16, at least max_connections*2, 8KB each
< #work_mem = 1024        # min 64, size in KB
< #maintenance_work_mem = 16384    # min 1024, size in KB
< #max_stack_depth = 2048        # min 100, size in KB
<
< #vacuum_cost_page_hit = 1    # 0-10000 credits
< #vacuum_cost_page_miss = 10    # 0-10000 credits
< #vacuum_cost_page_dirty = 20    # 0-10000 credits
< #vacuum_cost_limit = 200    # 0-10000 credits
< #vacuum_cost_naptime = 50    # 0-1000 milliseconds
---
> shared_buffers = 1000        # min 16, at least max_connections*2, 8KB each
> work_mem = 1024        # min 64, size in KB
> maintenance_work_mem = 16384    # min 1024, size in KB
> max_stack_depth = 2048        # min 100, size in KB
>
> vacuum_cost_page_hit = 1    # 0-10000 credits
> vacuum_cost_page_miss = 10    # 0-10000 credits
> vacuum_cost_page_dirty = 20    # 0-10000 credits
> vacuum_cost_limit = 200    # 0-10000 credits
> vacuum_cost_naptime = 50    # 0-1000 milliseconds
80,82c79,81
< #bgwriter_delay = 200        # 10-5000 milliseconds
< #bgwriter_percent = 1        # 0-100% of dirty buffers
< #bgwriter_maxpages = 100    # 1-1000 buffers max at once
---
> bgwriter_delay = 200        # 10-5000 milliseconds
> bgwriter_percent = 1        # 0-100% of dirty buffers
> bgwriter_maxpages = 100    # 1-1000 buffers max at once
86,87c85,86
< #max_fsm_pages = 20000        # min max_fsm_relations*16, 6 bytes each
< #max_fsm_relations = 1000    # min 100, ~50 bytes each
---
> max_fsm_pages = 20000        # min max_fsm_relations*16, 6 bytes each
> max_fsm_relations = 1000    # min 100, ~50 bytes each
91,92c90,91
< #max_files_per_process = 1000    # min 25
< #preload_libraries = ''
---
> max_files_per_process = 1000    # min 25
> preload_libraries = ''
101,102c100,101
< #fsync = true            # turns forced synchronization on or off
< #wal_sync_method = fsync    # the default varies across platforms:
---
> fsync = true            # turns forced synchronization on or off
> wal_sync_method = fsync    # the default varies across platforms:
104c103
< #wal_buffers = 8        # min 4, 8KB each
---
> wal_buffers = 8        # min 4, 8KB each
108,112c107,111
< #checkpoint_segments = 3    # in logfile segments, min 1, 16MB each
< #checkpoint_timeout = 300    # range 30-3600, in seconds
< #checkpoint_warning = 30    # 0 is off, in seconds
< #commit_delay = 0        # range 0-100000, in microseconds
< #commit_siblings = 5        # range 1-1000
---
> checkpoint_segments = 3    # in logfile segments, min 1, 16MB each
> checkpoint_timeout = 300    # range 30-3600, in seconds
> checkpoint_warning = 30    # 0 is off, in seconds
> commit_delay = 0        # range 0-100000, in microseconds
> commit_siblings = 5        # range 1-1000
121,128c120,127
< #enable_hashagg = true
< #enable_hashjoin = true
< #enable_indexscan = true
< #enable_mergejoin = true
< #enable_nestloop = true
< #enable_seqscan = true
< #enable_sort = true
< #enable_tidscan = true
---
> enable_hashagg = true
> enable_hashjoin = true
> enable_indexscan = true
> enable_mergejoin = true
> enable_nestloop = true
> enable_seqscan = true
> enable_sort = true
> enable_tidscan = true
132,136c131,135
< #effective_cache_size = 1000    # typically 8KB each
< #random_page_cost = 4        # units are one sequential page fetch cost
< #cpu_tuple_cost = 0.01        # (same)
< #cpu_index_tuple_cost = 0.001    # (same)
< #cpu_operator_cost = 0.0025    # (same)
---
> effective_cache_size = 1000    # typically 8KB each
> random_page_cost = 4        # units are one sequential page fetch cost
> cpu_tuple_cost = 0.01        # (same)
> cpu_index_tuple_cost = 0.001    # (same)
> cpu_operator_cost = 0.0025    # (same)
140,145c139,144
< #geqo = true
< #geqo_threshold = 12
< #geqo_effort = 5        # range 1-10
< #geqo_pool_size = 0        # selects default based on effort
< #geqo_generations = 0        # selects default based on effort
< #geqo_selection_bias = 2.0    # range 1.5-2.0
---
> geqo = true
> geqo_threshold = 12
> geqo_effort = 5        # range 1-10
> geqo_pool_size = 0        # selects default based on effort
> geqo_generations = 0        # selects default based on effort
> geqo_selection_bias = 2.0    # range 1.5-2.0
149,151c148,150
< #default_statistics_target = 10    # range 1-1000
< #from_collapse_limit = 8
< #join_collapse_limit = 8    # 1 disables collapsing of explicit JOINs
---
> default_statistics_target = 10    # range 1-1000
> from_collapse_limit = 8
> join_collapse_limit = 8    # 1 disables collapsing of explicit JOINs
160c159
< #log_destination = 'stderr'    # Valid values are combinations of stderr,
---
> log_destination = 'stderr'    # Valid values are combinations of stderr,
163,164c162,163
< #syslog_facility = 'LOCAL0'
< #syslog_ident = 'postgres'
---
> syslog_facility = 'LOCAL0'
> syslog_ident = 'postgres'
168c167
< #client_min_messages = notice    # Values, in order of decreasing detail:
---
> client_min_messages = notice    # Values, in order of decreasing detail:
172c171
< #log_min_messages = notice    # Values, in order of decreasing detail:
---
> log_min_messages = notice    # Values, in order of decreasing detail:
177c176
< #log_error_verbosity = default    # terse, default, or verbose messages
---
> log_error_verbosity = default    # terse, default, or verbose messages
179c178
< #log_min_error_statement = panic # Values in order of increasing severity:
---
> log_min_error_statement = panic # Values in order of increasing severity:
183c182
< #log_min_duration_statement = -1 # -1 is disabled, in milliseconds.
---
> log_min_duration_statement = -1 # -1 is disabled, in milliseconds.
185c184
< #silent_mode = false         # DO NOT USE without Syslog!
---
> silent_mode = false         # DO NOT USE without Syslog!
189,196c188,195
< #debug_print_parse = false
< #debug_print_rewritten = false
< #debug_print_plan = false
< #debug_pretty_print = false
< #log_connections = false
< #log_disconnections = false
< #log_duration = false
< #log_line_prefix = ''        # e.g. '<%u%%%d> '
---
> debug_print_parse = false
> debug_print_rewritten = false
> debug_print_plan = false
> debug_pretty_print = false
> log_connections = false
> log_disconnections = false
> log_duration = false
> log_line_prefix = ''        # e.g. '<%u%%%d> '
204,205c203,204
< #log_statement = 'none'        # none, mod, ddl, all
< #log_hostname = false
---
> log_statement = 'none'        # none, mod, ddl, all
> log_hostname = false
214,217c213,216
< #log_parser_stats = false
< #log_planner_stats = false
< #log_executor_stats = false
< #log_statement_stats = false
---
> log_parser_stats = false
> log_planner_stats = false
> log_executor_stats = false
> log_statement_stats = false
219c218
< #debug_shared_buffers = 0    # 0-600 seconds
---
> debug_shared_buffers = 0    # 0-600 seconds
223,227c222,226
< #stats_start_collector = true
< #stats_command_string = false
< #stats_block_level = false
< #stats_row_level = false
< #stats_reset_on_server_start = true
---
> stats_start_collector = true
> stats_command_string = false
> stats_block_level = false
> stats_row_level = false
> stats_reset_on_server_start = true
236,240c235,239
< #search_path = '$user,public'    # schema names
< #check_function_bodies = true
< #default_transaction_isolation = 'read committed'
< #default_transaction_read_only = false
< #statement_timeout = 0        # 0 is disabled, in milliseconds
---
> search_path = '$user,public'    # schema names
> check_function_bodies = true
> default_transaction_isolation = 'read committed'
> default_transaction_read_only = false
> statement_timeout = 0        # 0 is disabled, in milliseconds
244,248c243,247
< #datestyle = 'iso, mdy'
< #timezone = unknown        # actually, defaults to TZ environment setting
< #australian_timezones = false
< #extra_float_digits = 0        # min -15, max 2
< #client_encoding = sql_ascii    # actually, defaults to database encoding
---
> datestyle = 'iso, mdy'
> timezone = unknown        # actually, defaults to TZ environment setting
> australian_timezones = false
> extra_float_digits = 0        # min -15, max 2
> client_encoding = sql_ascii    # actually, defaults to database encoding
251,254c250,253
< #lc_messages = 'C'        # locale for system error message strings
< #lc_monetary = 'C'        # locale for monetary formatting
< #lc_numeric = 'C'        # locale for number formatting
< #lc_time = 'C'            # locale for time formatting
---
> lc_messages = 'C'        # locale for system error message strings
> lc_monetary = 'C'        # locale for monetary formatting
> lc_numeric = 'C'        # locale for number formatting
> lc_time = 'C'            # locale for time formatting
258,259c257,258
< #explain_pretty_print = true
< #dynamic_library_path = '$libdir'
---
> explain_pretty_print = true
> dynamic_library_path = '$libdir'
266,267c265,266
< #deadlock_timeout = 1000    # in milliseconds
< #max_locks_per_transaction = 64    # min 10, ~260*max_connections bytes each
---
> deadlock_timeout = 1000    # in milliseconds
> max_locks_per_transaction = 64    # min 10, ~260*max_connections bytes each
276,279c275,278
< #add_missing_from = true
< #regex_flavor = advanced    # advanced, extended, or basic
< #sql_inheritance = true
< #default_with_oids = true
---
> add_missing_from = true
> regex_flavor = advanced    # advanced, extended, or basic
> sql_inheritance = true
> default_with_oids = true
283c282,283
< #transform_null_equals = false
---
> transform_null_equals = false
>






-----BEGIN PGP SIGNATURE-----

iD8DBQFA8zJpvJuQZxSWSsgRAqZfAJ9lqog0A7CRgc+5h+qgavIAcSBWPQCfbrFj
gVEvRUYFY4BworLIukHm+s8=
=U5Nl
-----END PGP SIGNATURE-----



Re: Remove confusing commented-defaullts from postgresql.conf

From
Bruce Momjian
Date:
Uh, it is my understanding that all lines can't be uncommented.  Some of
the lines have defaults that are computed, like fsync_method.

Anyway, we are too busy to consider this for 7.5 unless someone wants to
research all these issues.

---------------------------------------------------------------------------

Greg Sabino Mullane wrote:
[ There is text before PGP section. ]
>
[ PGP not available, raw data follows ]
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> NotDashEscaped: You need GnuPG to verify this message
>
>
> I know we never reached a full consensus on hackers about the
> full solution to the postgresql.conf file, but I think everyone
> agreed that having commented-out "default" values was a bad thing,
> which caused confusion among even seasoned pg people. Thus, the
> patch below which simply removes the starting '#' from all settings.
> Forgive the lack of -c to the diff: I figured saving space was
> more important as the context is not needed in this case.
>
> --
> Greg Sabino Mullane greg@turnstep.com
> PGP Key: 0x14964AC8 200407122051
>
>
>
>
> Index: postgresql.conf.sample
> ===================================================================
> RCS file: /projects/cvsroot/pgsql-server/src/backend/utils/misc/postgresql.conf.sample,v
> retrieving revision 1.115
> diff -r1.115 postgresql.conf.sample
> 5c5
> < # This file consists of lines of the form:
> ---
> > # This file consists of lines of the form: GREG
> 11,12c11
> < # allowed values can be found in the PostgreSQL documentation. The
> < # commented-out settings shown in this file represent the default values.
> ---
> > # allowed values can be found in the PostgreSQL documentation.
> 28,31c27,30
> < # pgdata = '/usr/local/pgsql/data'        # use data in another directory
> < # hba_conf = '/etc/pgsql/pg_hba.conf'         # use hba info in another directory
> < # ident_conf = '/etc/pgsql/pg_ident.conf'    # use ident info in another directory
> < # external_pidfile= '/var/run/postgresql.pid'    # write an extra pid file
> ---
> > pgdata = '/usr/local/pgsql/data'        # use data in another directory
> > hba_conf = '/etc/pgsql/pg_hba.conf'         # use hba info in another directory
> > ident_conf = '/etc/pgsql/pg_ident.conf'    # use ident info in another directory
> > external_pidfile= '/var/run/postgresql.pid'    # write an extra pid file
> 40c39
> < #listen_addresses = 'localhost'    # what IP interface(s) to listen on;
> ---
> > listen_addresses = 'localhost'    # what IP interface(s) to listen on;
> 42,43c41,42
> < #port = 5432
> < #max_connections = 100
> ---
> > port = 5432
> > max_connections = 100
> 47,51c46,50
> < #superuser_reserved_connections = 2
> < #unix_socket_directory = ''
> < #unix_socket_group = ''
> < #unix_socket_permissions = 0777    # octal
> < #rendezvous_name = ''        # defaults to the computer name
> ---
> > superuser_reserved_connections = 2
> > unix_socket_directory = ''
> > unix_socket_group = ''
> > unix_socket_permissions = 0777    # octal
> > rendezvous_name = ''        # defaults to the computer name
> 55,59c54,58
> < #authentication_timeout = 60    # 1-600, in seconds
> < #ssl = false
> < #password_encryption = true
> < #krb_server_keyfile = ''
> < #db_user_namespace = false
> ---
> > authentication_timeout = 60    # 1-600, in seconds
> > ssl = false
> > password_encryption = true
> > krb_server_keyfile = ''
> > db_user_namespace = false
> 68,77c67,76
> < #shared_buffers = 1000        # min 16, at least max_connections*2, 8KB each
> < #work_mem = 1024        # min 64, size in KB
> < #maintenance_work_mem = 16384    # min 1024, size in KB
> < #max_stack_depth = 2048        # min 100, size in KB
> <
> < #vacuum_cost_page_hit = 1    # 0-10000 credits
> < #vacuum_cost_page_miss = 10    # 0-10000 credits
> < #vacuum_cost_page_dirty = 20    # 0-10000 credits
> < #vacuum_cost_limit = 200    # 0-10000 credits
> < #vacuum_cost_naptime = 50    # 0-1000 milliseconds
> ---
> > shared_buffers = 1000        # min 16, at least max_connections*2, 8KB each
> > work_mem = 1024        # min 64, size in KB
> > maintenance_work_mem = 16384    # min 1024, size in KB
> > max_stack_depth = 2048        # min 100, size in KB
> >
> > vacuum_cost_page_hit = 1    # 0-10000 credits
> > vacuum_cost_page_miss = 10    # 0-10000 credits
> > vacuum_cost_page_dirty = 20    # 0-10000 credits
> > vacuum_cost_limit = 200    # 0-10000 credits
> > vacuum_cost_naptime = 50    # 0-1000 milliseconds
> 80,82c79,81
> < #bgwriter_delay = 200        # 10-5000 milliseconds
> < #bgwriter_percent = 1        # 0-100% of dirty buffers
> < #bgwriter_maxpages = 100    # 1-1000 buffers max at once
> ---
> > bgwriter_delay = 200        # 10-5000 milliseconds
> > bgwriter_percent = 1        # 0-100% of dirty buffers
> > bgwriter_maxpages = 100    # 1-1000 buffers max at once
> 86,87c85,86
> < #max_fsm_pages = 20000        # min max_fsm_relations*16, 6 bytes each
> < #max_fsm_relations = 1000    # min 100, ~50 bytes each
> ---
> > max_fsm_pages = 20000        # min max_fsm_relations*16, 6 bytes each
> > max_fsm_relations = 1000    # min 100, ~50 bytes each
> 91,92c90,91
> < #max_files_per_process = 1000    # min 25
> < #preload_libraries = ''
> ---
> > max_files_per_process = 1000    # min 25
> > preload_libraries = ''
> 101,102c100,101
> < #fsync = true            # turns forced synchronization on or off
> < #wal_sync_method = fsync    # the default varies across platforms:
> ---
> > fsync = true            # turns forced synchronization on or off
> > wal_sync_method = fsync    # the default varies across platforms:
> 104c103
> < #wal_buffers = 8        # min 4, 8KB each
> ---
> > wal_buffers = 8        # min 4, 8KB each
> 108,112c107,111
> < #checkpoint_segments = 3    # in logfile segments, min 1, 16MB each
> < #checkpoint_timeout = 300    # range 30-3600, in seconds
> < #checkpoint_warning = 30    # 0 is off, in seconds
> < #commit_delay = 0        # range 0-100000, in microseconds
> < #commit_siblings = 5        # range 1-1000
> ---
> > checkpoint_segments = 3    # in logfile segments, min 1, 16MB each
> > checkpoint_timeout = 300    # range 30-3600, in seconds
> > checkpoint_warning = 30    # 0 is off, in seconds
> > commit_delay = 0        # range 0-100000, in microseconds
> > commit_siblings = 5        # range 1-1000
> 121,128c120,127
> < #enable_hashagg = true
> < #enable_hashjoin = true
> < #enable_indexscan = true
> < #enable_mergejoin = true
> < #enable_nestloop = true
> < #enable_seqscan = true
> < #enable_sort = true
> < #enable_tidscan = true
> ---
> > enable_hashagg = true
> > enable_hashjoin = true
> > enable_indexscan = true
> > enable_mergejoin = true
> > enable_nestloop = true
> > enable_seqscan = true
> > enable_sort = true
> > enable_tidscan = true
> 132,136c131,135
> < #effective_cache_size = 1000    # typically 8KB each
> < #random_page_cost = 4        # units are one sequential page fetch cost
> < #cpu_tuple_cost = 0.01        # (same)
> < #cpu_index_tuple_cost = 0.001    # (same)
> < #cpu_operator_cost = 0.0025    # (same)
> ---
> > effective_cache_size = 1000    # typically 8KB each
> > random_page_cost = 4        # units are one sequential page fetch cost
> > cpu_tuple_cost = 0.01        # (same)
> > cpu_index_tuple_cost = 0.001    # (same)
> > cpu_operator_cost = 0.0025    # (same)
> 140,145c139,144
> < #geqo = true
> < #geqo_threshold = 12
> < #geqo_effort = 5        # range 1-10
> < #geqo_pool_size = 0        # selects default based on effort
> < #geqo_generations = 0        # selects default based on effort
> < #geqo_selection_bias = 2.0    # range 1.5-2.0
> ---
> > geqo = true
> > geqo_threshold = 12
> > geqo_effort = 5        # range 1-10
> > geqo_pool_size = 0        # selects default based on effort
> > geqo_generations = 0        # selects default based on effort
> > geqo_selection_bias = 2.0    # range 1.5-2.0
> 149,151c148,150
> < #default_statistics_target = 10    # range 1-1000
> < #from_collapse_limit = 8
> < #join_collapse_limit = 8    # 1 disables collapsing of explicit JOINs
> ---
> > default_statistics_target = 10    # range 1-1000
> > from_collapse_limit = 8
> > join_collapse_limit = 8    # 1 disables collapsing of explicit JOINs
> 160c159
> < #log_destination = 'stderr'    # Valid values are combinations of stderr,
> ---
> > log_destination = 'stderr'    # Valid values are combinations of stderr,
> 163,164c162,163
> < #syslog_facility = 'LOCAL0'
> < #syslog_ident = 'postgres'
> ---
> > syslog_facility = 'LOCAL0'
> > syslog_ident = 'postgres'
> 168c167
> < #client_min_messages = notice    # Values, in order of decreasing detail:
> ---
> > client_min_messages = notice    # Values, in order of decreasing detail:
> 172c171
> < #log_min_messages = notice    # Values, in order of decreasing detail:
> ---
> > log_min_messages = notice    # Values, in order of decreasing detail:
> 177c176
> < #log_error_verbosity = default    # terse, default, or verbose messages
> ---
> > log_error_verbosity = default    # terse, default, or verbose messages
> 179c178
> < #log_min_error_statement = panic # Values in order of increasing severity:
> ---
> > log_min_error_statement = panic # Values in order of increasing severity:
> 183c182
> < #log_min_duration_statement = -1 # -1 is disabled, in milliseconds.
> ---
> > log_min_duration_statement = -1 # -1 is disabled, in milliseconds.
> 185c184
> < #silent_mode = false         # DO NOT USE without Syslog!
> ---
> > silent_mode = false         # DO NOT USE without Syslog!
> 189,196c188,195
> < #debug_print_parse = false
> < #debug_print_rewritten = false
> < #debug_print_plan = false
> < #debug_pretty_print = false
> < #log_connections = false
> < #log_disconnections = false
> < #log_duration = false
> < #log_line_prefix = ''        # e.g. '<%u%%%d> '
> ---
> > debug_print_parse = false
> > debug_print_rewritten = false
> > debug_print_plan = false
> > debug_pretty_print = false
> > log_connections = false
> > log_disconnections = false
> > log_duration = false
> > log_line_prefix = ''        # e.g. '<%u%%%d> '
> 204,205c203,204
> < #log_statement = 'none'        # none, mod, ddl, all
> < #log_hostname = false
> ---
> > log_statement = 'none'        # none, mod, ddl, all
> > log_hostname = false
> 214,217c213,216
> < #log_parser_stats = false
> < #log_planner_stats = false
> < #log_executor_stats = false
> < #log_statement_stats = false
> ---
> > log_parser_stats = false
> > log_planner_stats = false
> > log_executor_stats = false
> > log_statement_stats = false
> 219c218
> < #debug_shared_buffers = 0    # 0-600 seconds
> ---
> > debug_shared_buffers = 0    # 0-600 seconds
> 223,227c222,226
> < #stats_start_collector = true
> < #stats_command_string = false
> < #stats_block_level = false
> < #stats_row_level = false
> < #stats_reset_on_server_start = true
> ---
> > stats_start_collector = true
> > stats_command_string = false
> > stats_block_level = false
> > stats_row_level = false
> > stats_reset_on_server_start = true
> 236,240c235,239
> < #search_path = '$user,public'    # schema names
> < #check_function_bodies = true
> < #default_transaction_isolation = 'read committed'
> < #default_transaction_read_only = false
> < #statement_timeout = 0        # 0 is disabled, in milliseconds
> ---
> > search_path = '$user,public'    # schema names
> > check_function_bodies = true
> > default_transaction_isolation = 'read committed'
> > default_transaction_read_only = false
> > statement_timeout = 0        # 0 is disabled, in milliseconds
> 244,248c243,247
> < #datestyle = 'iso, mdy'
> < #timezone = unknown        # actually, defaults to TZ environment setting
> < #australian_timezones = false
> < #extra_float_digits = 0        # min -15, max 2
> < #client_encoding = sql_ascii    # actually, defaults to database encoding
> ---
> > datestyle = 'iso, mdy'
> > timezone = unknown        # actually, defaults to TZ environment setting
> > australian_timezones = false
> > extra_float_digits = 0        # min -15, max 2
> > client_encoding = sql_ascii    # actually, defaults to database encoding
> 251,254c250,253
> < #lc_messages = 'C'        # locale for system error message strings
> < #lc_monetary = 'C'        # locale for monetary formatting
> < #lc_numeric = 'C'        # locale for number formatting
> < #lc_time = 'C'            # locale for time formatting
> ---
> > lc_messages = 'C'        # locale for system error message strings
> > lc_monetary = 'C'        # locale for monetary formatting
> > lc_numeric = 'C'        # locale for number formatting
> > lc_time = 'C'            # locale for time formatting
> 258,259c257,258
> < #explain_pretty_print = true
> < #dynamic_library_path = '$libdir'
> ---
> > explain_pretty_print = true
> > dynamic_library_path = '$libdir'
> 266,267c265,266
> < #deadlock_timeout = 1000    # in milliseconds
> < #max_locks_per_transaction = 64    # min 10, ~260*max_connections bytes each
> ---
> > deadlock_timeout = 1000    # in milliseconds
> > max_locks_per_transaction = 64    # min 10, ~260*max_connections bytes each
> 276,279c275,278
> < #add_missing_from = true
> < #regex_flavor = advanced    # advanced, extended, or basic
> < #sql_inheritance = true
> < #default_with_oids = true
> ---
> > add_missing_from = true
> > regex_flavor = advanced    # advanced, extended, or basic
> > sql_inheritance = true
> > default_with_oids = true
> 283c282,283
> < #transform_null_equals = false
> ---
> > transform_null_equals = false
> >
>
>
>
>
>
>
> -----BEGIN PGP SIGNATURE-----
>
> iD8DBQFA8zJpvJuQZxSWSsgRAqZfAJ9lqog0A7CRgc+5h+qgavIAcSBWPQCfbrFj
> gVEvRUYFY4BworLIukHm+s8=
> =U5Nl
> -----END PGP SIGNATURE-----
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: don't forget to increase your free space map settings
>
[ Decrypting message... End of raw data. ]

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

Re: Remove confusing commented-defaullts from postgresql.conf

From
Tom Lane
Date:
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Uh, it is my understanding that all lines can't be uncommented.  Some of
> the lines have defaults that are computed, like fsync_method.

Yes, fsync_method shouldn't be uncommented unless you're prepared to put
in the same default value presently computed in xlog.c.  (Now that
initdb is in C, it might not be unreasonable to move the #ifdef hacking
that's in xlog.c over to initdb...)

Also you'd need to make some other adjustments in initdb; IIRC it does
sed-like substitutions that will break if the input is not just so.
I don't recall at the moment but there may be some things of the same
ilk done during "make install".  And check the documentation to see if
any discussion needs updated.

> Anyway, we are too busy to consider this for 7.5 unless someone wants to
> research all these issues.

No reason Greg can't do it if he wants to ... but the .sample file
itself is certainly not all that needs changing.

            regards, tom lane

Re: Remove confusing commented-defaullts from postgresql.conf

From
"Greg Sabino Mullane"
Date:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


> No reason Greg can't do it if he wants to ... but the .sample file
> itself is certainly not all that needs changing.

No problem. I didn't expect the patch to pass as is, just wanted
to get my foot in the door. :)

- --
Greg Sabino Mullane greg@turnstep.com
PGP Key: 0x14964AC8 200407130745
-----BEGIN PGP SIGNATURE-----

iD8DBQFA88txvJuQZxSWSsgRAtVJAKD8oWEHqudEOQAw5H4aZM9xYmBEJQCgriKb
gdWb6Xkpfy9Dyy837TJ+to8=
=z0P0
-----END PGP SIGNATURE-----



Re: Remove confusing commented-defaullts from postgresql.conf

From
Bruce Momjian
Date:
Greg Sabino Mullane wrote:
[ There is text before PGP section. ]
>
[ PGP not available, raw data follows ]
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
>
> > No reason Greg can't do it if he wants to ... but the .sample file
> > itself is certainly not all that needs changing.
>
> No problem. I didn't expect the patch to pass as is, just wanted
> to get my foot in the door. :)

However, this has to be considered a feature addition after feature
freeze so I am not sure we are going to do this for 7.5.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073