babelfish-for-postgresql / babelfish_extensions / 21711016157

76%

BABEL_6_X_DEV: 78%

Build Type

push

github

Committed by web-flow

Commit Message

Replaced redundant plan generation in execsql for pltsql statements (#4425) (#4531)

Currently, to determine the type of the statement and whether it has a
RETURNING clause, a plan is generated by calling SPI_plan_get_cached_plan().
This leads to additional overhead. This commit removes the call, instead
using the query list to determine the type.

There were multiple dependencies on the plan being generated early, these
have been changed to instead walk through query trees.

Engine PR: https://github.com/babelfish-for-postgresql/postgresql_modified_for_babelfish/pull/701

Task: BABEL-3756

Authored by: Saurav Samantaray <sauravay@amazon.com>

Coverage Stats

43 of 44 new or added lines in 2 files covered. (97.73%)

937 existing lines in 3 files now uncovered.

50702 of 66442 relevant lines covered (76.31%)

392758.5 hits per line