From: | John Gray <jgray(at)azuli(dot)co(dot)uk> |
---|---|
To: | Rod Taylor <rbt(at)zort(dot)ca> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Christopher Kings-Lynne <chriskl(at)familyhealth(dot)com(dot)au>, Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: command.c breakup |
Date: | 2002-04-14 20:43:09 |
Message-ID: | 1018816992.1375.48.camel@adzuki |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, 2002-04-14 at 21:30, Rod Taylor wrote:
> I'm not exactly sure what you're touching, but could it wait for the
> below pg_depend patch to be either accepted or rejected? It lightly
> fiddles with a number of files in the command and catalog directories.
>
> http://archives.postgresql.org/pgsql-patches/2002-04/msg00050.php
>
Well, I'm working on it now and it's about 75% done. I hope to post the
patch within the next few hours. I'm sorry that I wasn't aware of your
patch -but commands/ is a busy place at present :). I've scanned your
patch very briefly and the major impacts I can see are:
1) The ALTER TABLE code will be in tablecmds.c (but exactly the same
code as at present)
2) The type support will be in typecmds.c (define.c and remove.c are
essentially gone -the define and remove commands for foo are in general
now together in foocmds.c
I'm not touching anything in the catalog directory.
Note that as I'm only shuffling code from one file to another, your
patch shouldn't need much modification to get it working afterwards -
although there is an intention to tidy up common code in the commands/
directory as a second phase, this will consist of more "ordinary"
patches...
Regards
John
From | Date | Subject | |
---|---|---|---|
Next Message | Rod Taylor | 2002-04-14 20:58:40 | Re: command.c breakup |
Previous Message | Karel Zak | 2002-04-14 20:39:32 | Re: experimental pg_qcache patch |