Generally, the patch looks fine. There are few issues still: - plpgsql: the result columns _do_ create local variables. AIUI, they should not? - pg_dump: is the psql_assert() introduction necessary, considering it is used only in one place? - There should be regression test for plpgsql too, that test if the behaviour is correct. - The documentation should mention behaviour difference from OUT parameters. Wishlist (probably out of scope for this patch): - plpgsql: a way to create record variable for result row. Something like: CREATE FUNCTION foo(..) RETURNS TABLE (..) AS $$ DECLARE retval foo%ROWTYPE; Currently the OUT parameters are quite painful to use due to bad name resolving logic. Such feature would be perfect replacement. -- marko
Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера