babelfish-for-postgresql / babelfish_extensions / 21701959476

78%

BABEL_6_X_DEV: 78%

push

github

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

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>

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

52717 of 68471 relevant lines covered (76.99%)

387893.88 hits per line