Re: ALT*R T*BL* S*T / DR*P N*T N*LL patch - 4th try

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: ALT*R T*BL* S*T / DR*P N*T N*LL patch - 4th try
Дата
Msg-id 3201.1017519636@sss.pgh.pa.us
обсуждение исходный текст
Ответ на ALT*R T*BL* S*T / DR*P N*T N*LL patch - 4th try  ("Christopher Kings-Lynne" <chriskl@familyhealth.com.au>)
Ответы Re: ALT*R T*BL* S*T / DR*P N*T N*LL patch - 4th try  (Christopher Kings-Lynne <chriskl@familyhealth.com.au>)
Список pgsql-patches
"Christopher Kings-Lynne" <chriskl@familyhealth.com.au> writes:
> This is a complete patch to implement changing the nullability of an
> attribute.

This looks good but I'm afraid it's out of step with recent changes
for schema support.  In particular, heap_openr is now The Wrong Thing
to use for anything except access to system relations.  As of CVS tip,
all the AlterTable routines use OID parameters not name parameters
to identify the table to work on.  Could I ask you to update the patch
to match the coding style in CVS tip?

> The only thing I haven't checked are my ecpg changes.  I would like someone
> with more ecpg experience to check my preproc.y changes.

I'd suggest leaving ecpg to Michael; he seems to prefer to do those
updates himself.

> Since I have now added two new large functions to command.c, I propose that
> sometime before 7.3 beta, command.c is refactored and an alter.c created.
> There is lots of common code in the Alter* functions that should be reused.

This is definitely a good thing to do.  I think someone else (John
Gray?) had already volunteered, but I have not seen a patch go by.

            regards, tom lane

В списке pgsql-patches по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Insert ... (a, b, DEFAULT, c, d);
Следующее
От: "Rod Taylor"
Дата:
Сообщение: Re: Insert ... (a, b, DEFAULT, c, d);