babelfish-for-postgresql / babelfish_extensions / 18138152024

76%

BABEL_5_X_DEV: 77%

Build Type

push

github

Committed by web-flow

Commit Message

Fix permission checks for views in Planner phase for cross-db cases (#4100)

Introduced the permission checks on view in the planner phase. In
Babelfish there is permission specific logic which is implemented in
executor_hook to support the query on cross-db views. While implementing
cross-db view query support, We assumed that permission checks always
happens at executor. After mentioned community changes that assumption
got broken and cross-db view queries throwing errors of insufficient
privileges.

This commit fixes above issue by adding same logic for supporting
cross-db view query to pre-planning phase using pltsql_planner_hook. It
is basically walking through whole query tree and inserts checkAsUser
field for all the view RTE, so that permission check on views always
happens on TSQL login to support cross-db usecases.

Issues Resolved: BABEL-6035
Signed-off-by: Harsh Lunagariya <lunharsh@amazon.com>

Run Details

26 of 26 new or added lines in 1 file covered. (100.0%)

491 existing lines in 5 files now uncovered.

49727 of 65714 relevant lines covered (75.67%)

322743.76 hits per line