On Tue, Sep 28, 2010 at 13:50, Robert Haas <robertmhaas@gmail.com> wrote:
> On Tue, Sep 28, 2010 at 3:57 AM, Shigeru HANADA
> <hanada@metrosystems.co.jp> wrote:
>> On Mon, 27 Sep 2010 21:07:33 -0400
>> Robert Haas <robertmhaas@gmail.com> wrote:
>>> I found and fixed a few more issues and committed this. The pg_dump
>>> support had a few escaping bugs, and I added tab completion support
>>> for psql. Considering the size of the patch, it seems likely that
>>> there are some issues we both overlooked, but this is as solid as I
>>> can make it for right now.
>> Some OIDs used in SECURITY LABEL patch have already been used for
>> some functions such as pg_stat_get_xact_numscans().
>>
>> The src/include/catalog/duplicate_oids script reports that 3037 ~
>> 3040 are used two or more times.
>>
>> Though all regression tests finish successfully, should this be
>> fixed ?
>
> Woops. Thanks for the report, fixed. I wish we had a regression test
> that would catch these mistakes. It's easy to forget to run this
> script.
Could we run the script as part of the regression tests? :-)
Or if not, could we have the buildfarm run it?
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/