From 02363c52d0d03f58b8e79088a7261a9fc1e3bbb7 Mon Sep 17 00:00:00 2001 From: Pierre Date: Fri, 31 Jan 2020 23:58:55 +0100 Subject: [PATCH 2/2] Add support for array contains in dependency check --- src/backend/statistics/dependencies.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/backend/statistics/dependencies.c b/src/backend/statistics/dependencies.c index d4844a9ec6..c3f37a474b 100644 --- a/src/backend/statistics/dependencies.c +++ b/src/backend/statistics/dependencies.c @@ -785,7 +785,7 @@ dependency_is_compatible_clause(Node *clause, Index relid, AttrNumber *attnum) return false; /* - * If it's not an "=" operator, just ignore the clause, as it's not + * If it's not an "=" or "@>" operator, just ignore the clause, as it's not * compatible with functional dependencies. * * This uses the function for estimating selectivity, not the operator @@ -796,8 +796,13 @@ dependency_is_compatible_clause(Node *clause, Index relid, AttrNumber *attnum) * selectivity functions, and to be more consistent with decisions * elsewhere in the planner. */ - if (get_oprrest(expr->opno) != F_EQSEL) - return false; + switch(get_oprrest(expr->opno)) { + case F_EQSEL: + case F_ARRAYCONTSEL: + break; + default: + return false; + } /* OK to proceed with checking "var" */ } -- 2.24.1