On Tue, Sep 28, 2010 at 13:50, Robert Haas <robertmh...@gmail.com> wrote: > On Tue, Sep 28, 2010 at 3:57 AM, Shigeru HANADA > <han...@metrosystems.co.jp> wrote: >> On Mon, 27 Sep 2010 21:07:33 -0400 >> Robert Haas <robertmh...@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/ -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers