The attached patch is additional regression tests of ALTER FUNCTION with LEAKPROOF based on your patch. It also moves create_function_3 into the group with create_aggregate and so on.
Thanks, 2012/2/14 Kohei KaiGai <kai...@kaigai.gr.jp>: > 2012/2/14 Robert Haas <robertmh...@gmail.com>: >> On Tue, Feb 14, 2012 at 4:55 AM, Kohei KaiGai <kai...@kaigai.gr.jp> wrote: >>> I could not find out where is the origin of grammer conflicts, although >>> it does not conflict with any options within ALTER FUNCTION. >>> >>> Do you think the idea of ALTER ... NOT LEAKPROOF should be >>> integrated within v9.2 timeline also? >> >> Yes. Did you notice that I attached a patch to make that work? I'll >> commit that today or tomorrow unless someone comes up with a better >> solution. >> > Yes. I'll be available to work on the feature based on this patch. > It was a headache of mine to implement alter statement to add/remove > leakproof attribute. > >>>> I also think we ought to stick create_function_3 into one of the >>>> parallel groups in the regression tests, if possible. Can you >>>> investigate that? >>>> >>> Not yet. This test does not have dependency with other tests, >>> so, I'm optimistic to run create_function_3 concurrently. >> >> Me, too. >> > I tried to move create_function_3 into the group of create_view and > create_index, then it works correctly. > > Thanks, > -- > KaiGai Kohei <kai...@kaigai.gr.jp> -- KaiGai Kohei <kai...@kaigai.gr.jp>
pgsql-v9.2-alter-function-leakproof-regtest.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers