On 20 June 2011 11:00, Dave Page <dpage@pgadmin.org> wrote:
> On Fri, Jun 17, 2011 at 6:39 PM, Thom Brown <thom@linux.com> wrote:
>> On 17 June 2011 18:18, Thom Brown <thom@linux.com> wrote:
>>> On 17 June 2011 18:16, Thom Brown <thom@linux.com> wrote:
>>>
>>>> Very well then. Dodgy patch attached.
>>>
>>> And again without ugly whitespace.
>>
>> Erk... just been trying to break it and found that this generates the
>> alter statement every time once the value has been set, so fixed that
>> too (attached).
>
> LOL. The patch looks great as far as it goes - not at all dodgy. It
> currently only affects tables though - you need to do roughly the same
> thing in dlgIndex.cpp and dlgIndexConstraint.cpp to cover the cases
> you reported :-)
Okay, updated patch attached. This enables modification of fill
factor on existing tables and primary keys.
--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company