From 0f737fa8ec801921f305cd44ccf1c6727e371464 Mon Sep 17 00:00:00 2001 From: Tender Wang Date: Wed, 9 Oct 2024 15:10:44 +0800 Subject: [PATCH] Remove an unnecessary check as Var can only belong to one sied of joinrel. --- contrib/postgres_fdw/postgres_fdw.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/contrib/postgres_fdw/postgres_fdw.c b/contrib/postgres_fdw/postgres_fdw.c index 6beae0fa37f2..b2510c488218 100644 --- a/contrib/postgres_fdw/postgres_fdw.c +++ b/contrib/postgres_fdw/postgres_fdw.c @@ -5768,12 +5768,11 @@ semijoin_target_ok(PlannerInfo *root, RelOptInfo *joinrel, RelOptInfo *outerrel, if (!IsA(var, Var)) continue; - if (bms_is_member(var->varno, innerrel->relids) && - !bms_is_member(var->varno, outerrel->relids)) + if (bms_is_member(var->varno, innerrel->relids)) { /* * The planner can create semi-join, which refers to inner rel - * vars in its target list. However, we deparse semi-join as an + * vars in its target list. However, we deparse semi-join as an * exists() subquery, so can't handle references to inner rel in * the target list. */