Re: CF 2009-07: initial reviewing assignments

Поиск
Список
Период
Сортировка
От Dimitri Fontaine
Тема Re: CF 2009-07: initial reviewing assignments
Дата
Msg-id 56B176A2-F61F-4713-88C0-D60FB34869EB@hi-media.com
обсуждение исходный текст
Ответ на Re: CF 2009-07: initial reviewing assignments  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: CF 2009-07: initial reviewing assignments  (Dimitri Fontaine <dfontaine@hi-media.com>)
Список pgsql-rrreviewers
Le 18 juil. 09 à 22:57, Robert Haas a écrit :
> If you have time take a look at "Parser's hook based on FuncCall".  I
> think Tom has commented on this previously so I'm not sure how much
> reviewing remains to be done, but maybe you can bring things to a
> conclusion somehow.

I'll read the threads and the patch once I'm done with yours, then. Up
until then, it's free for anybody :)

>> Maybe I could even try to fix the bitrot myself, it doesn't look so
>> hard,
>> just lots of careful work for anyone who has never done it before...
>
> As far as I can see it's entirely whitespace changes.

This is mixed with your column addition, so I didn't see that. Now
I've just kept your version of the file, which solves the conflict:
merge is done, builds ok, first basic test means I have more work ahead:

     Table "pg_catalog.pg_attribute"
     Column     |   Type    | Modifiers
---------------+-----------+-----------
...
  attstattarget | integer   | not null
  attdistinct   | real      | not null
  attlen        | smallint  | not null

I'm fixing the status in commitfest application.

Regards,
--
dim

I'd better get some sleep now. :)

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

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: CF 2009-07: initial reviewing assignments
Следующее
От: Jeff Davis
Дата:
Сообщение: Re: CF 2009-07: initial reviewing assignments