From: | Dave Page <dpage(at)postgresql(dot)org> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Table FILLFACTOR support |
Date: | 2007-05-10 07:46:33 |
Message-ID: | 4642CDD9.5050704@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Guillaume Lelarge wrote:
> Hi all,
>
> Here is a patch to add support for FILLFACTOR option on tables.
> Fillfactor is visible on the properties tab of a table view and on the
> table dialog. A user can add a fillfactor value at table's creation
> time, but can't change it later.
>
> I don't think this patch should be applied right now. I need tips and
> advices on it. I also wish to add index FILLFACTOR option support if I
> still have time to.
OK. My only suggestion is don't mix up your operator usage - for
example, on one line you might use:
sql += wxT("Foo");
and on the next:
sql = sql + wxT("Foo");
Other than that, it passes the eyeball test - everything looks sane, and
you've hit the parts of the code I'd expect.
Please test table creation on 8.1 and 8.2, and check that the reverse
engineered SQL from a suitable table on each server version can be
executed as expected and go ahead and commit. Oh, and don't forget to
hit the CHANGELOG!
Thanks, Dave.
From | Date | Subject | |
---|---|---|---|
Next Message | svn | 2007-05-10 08:06:26 | SVN Commit by dpage: r6286 - trunk/pgadmin3/pgadmin/ui |
Previous Message | Dave Page | 2007-05-10 07:24:14 | Re: [Enter] key moves to the end of editing pane |