Re: fillfactor gets set to zero for toast tables - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: fillfactor gets set to zero for toast tables
Date
Msg-id 1275331853-sup-8934@alvh.no-ip.org
Whole thread Raw
In response to Re: fillfactor gets set to zero for toast tables  (Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: fillfactor gets set to zero for toast tables
List pgsql-hackers
Excerpts from Takahiro Itagaki's message of mié may 26 03:32:56 -0400 2010:
> 
> Alvaro Herrera <alvherre@commandprompt.com> wrote:
> 
> > Excerpts from Tom Lane's message of vie may 14 15:03:57 -0400 2010:
> > 
> > > Maybe a better solution is to have some kind of notion of a default-only
> > > entry, which is sufficient to insert the default into the struct but
> > > isn't accepted as a user-settable item.
> > 
> > This patch (for 8.4, but applies fuzzily to 9.0) implements this idea.
> > Note that there's no explicit check that every heap option has a
> > corresponding toast option; that's left to the developer's judgement to
> > add.  I added the new member to relopt_gen struct so that existing
> > entries did not require changes in initializers.
> 
> The new "default_only" field can be initialized only from the internal codes
> and is not exported to user definded reloptions. We could add an additional
> argument to add_xxx_reloption() functions, but it breaks ABI.

Do we really need default_only entries in user-defined reloptions?

We have yet to see any indication that anybody is using user-defined
reloptions at all ...  It'd be good to have an use case at least (if
only to ensure that the API we're providing is sufficient).

If in the future we determine that we need to offer user-defined
default_only reloptions, perhaps we can add new entry points in the
reloptions API.

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Keepalive for max_standby_delay
Next
From: Bruce Momjian
Date:
Subject: Re: pg_start_backup and pg_stop_backup Re: Re: [COMMITTERS] pgsql: Make CheckRequiredParameterValues() depend upon correct