Skip to content

Commit

Permalink
resowner removed from CachedPlanAllowsSimpleValidityCheck
Browse files Browse the repository at this point in the history
  • Loading branch information
Parikshit Sarode committed Oct 5, 2023
1 parent 46b3436 commit e310e1d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion contrib/babelfishpg_tsql/src/pl_exec.c
Original file line number Diff line number Diff line change
Expand Up @@ -7663,7 +7663,7 @@ exec_eval_simple_expr(PLtsql_execstate *estate,
* refcount on the new plan, stored in simple_eval_resowner.
*/
if (CachedPlanAllowsSimpleValidityCheck(expr->expr_simple_plansource,
cplan, CurrentResourceOwner) &&
cplan) &&
CachedPlanIsSimplyValid(expr->expr_simple_plansource, cplan,
estate->simple_eval_resowner))
{
Expand Down
2 changes: 1 addition & 1 deletion contrib/babelfishpg_tsql/src/prepare.c
Original file line number Diff line number Diff line change
Expand Up @@ -565,7 +565,7 @@ exec_simple_check_plan(PLtsql_execstate *estate, PLtsql_expr *expr)
* CachedPlanIsSimplyValid. Given the restrictions above, it's unlikely
* that this could fail, but if it does, just treat plan as not simple.
*/
if (CachedPlanAllowsSimpleValidityCheck(plansource, cplan, CurrentResourceOwner))
if (CachedPlanAllowsSimpleValidityCheck(plansource, cplan))
{
/*
* OK, use CachedPlanIsSimplyValid to save a refcount on the plan in
Expand Down

0 comments on commit e310e1d

Please sign in to comment.