babelfish-for-postgresql / babelfish_extensions / 15164143963

77%

push

github

Allow parallelism for pltsql return expression after community commit 556f7b7 (#3725)

With the commit 556f7b7, we unintentionally blocked parallelism to
evaluate plpgsql return expression because maxtuples = 2 is being passed
to exec_run_select(...) from exec_eval_expr(...) to evaluate the return
expression of the plpgsql function.

Idea to fix this issue to pass maxtuples = 0. It is safe to do it
because number of processed rows is anyway being checked later in
exec_eval_expr(...). But with this, there is not real caller remained
which calls exec_run_select(...) with maxtuple != 0 so updated definition
of exec_run_select(...) to remove maxtuples argument.

This commit also fixes parallel context leak in case of statement terminating 
error originating from PLTSQL function body. This is important because 
subsequent resetting search_path or table variable cleanup operations may
end up error.

Task: BABEL-5599
Signed-off-by: Dipesh Dhameliya <dipeshdhameliya125@gmail.com>

6 of 11 new or added lines in 3 files covered. (54.55%)

48572 of 64443 relevant lines covered (75.37%)

306339.43 hits per line