babelfish-for-postgresql / babelfish_extensions / 15208246028

76%

BABEL_5_X_DEV: 77%

Build Type

push

github

Committed by web-flow

Commit Message

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

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

(cherry picked from commit b3f148486)

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

Run Details

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

2 existing lines in 1 file now uncovered.

48416 of 64300 relevant lines covered (75.3%)

314419.28 hits per line