1
74%
BABEL_5_X_DEV: 77%

Ran 18 Sep 2025 07:21AM UTC

Files 115

Run time 5s

Badge

Embed ▾

Committed 18 Sep 2025 06:51AM UTC coverage: 74.199% (+0.01%) from 74.188%

Job # 17820546495.1

Build Type

push

github

Committed by

web-flow

Commit Message

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

Commit e2d4ef8de86 - "Fix security checks in selectivity estimation functions."
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 brocken 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>

cr: https://code.amazon.com/reviews/CR-216259370

Run Details

43951 of 59234 relevant lines covered (74.2%)

275757.53 hits per line

babelfish-for-postgresql / babelfish_extensions / 17820546495 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17820546495.1

babelfish-for-postgresql / babelfish_extensions / 17820546495 / 1 74% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17820546495.1

babelfish-for-postgresql / babelfish_extensions / 17820546495 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x