database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-07-30 11:03:19 +03:00
Code Activity

Allow non-btree unique indexes for matviews

Browse Source 
We were rejecting non-btree indexes in some cases owing to the
inability to determine the equality operators for other index AMs;
that problem no longer exists, because we can look up the equality
operator using COMPARE_EQ.

Stop rejecting these indexes, but instead rely on all unique indexes
having equality operators.  Unique indexes must have equality
operators.

Author: Mark Dilger <mark.dilger@enterprisedb.com>
Discussion: https://www.postgresql.org/message-id/flat/E72EAA49-354D-4C2E-8EB9-255197F55330@enterprisedb.com
This commit is contained in:
Peter Eisentraut
2025-03-18 11:29:15 +01:00
parent f278e1fe30
commit 9d6db8bec1
1 changed files with 4 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/backend/commands/matview.c
View File

@ -774,16 +774,14 @@ refresh_by_match_merge(Oid matviewOid, Oid tempOid, Oid relowner,
if (!HeapTupleIsValid(cla_ht)) if (!HeapTupleIsValid(cla_ht))
elog(ERROR, "cache lookup failed for opclass %u", opclass); elog(ERROR, "cache lookup failed for opclass %u", opclass);
cla_tup = (Form_pg_opclass) GETSTRUCT(cla_ht); cla_tup = (Form_pg_opclass) GETSTRUCT(cla_ht);
Assert(cla_tup->opcmethod == BTREE_AM_OID);
opfamily = cla_tup->opcfamily; opfamily = cla_tup->opcfamily;
opcintype = cla_tup->opcintype; opcintype = cla_tup->opcintype;
ReleaseSysCache(cla_ht); ReleaseSysCache(cla_ht);
op = get_opfamily_member(opfamily, opcintype, opcintype, op = get_opfamily_member_for_cmptype(opfamily, opcintype, opcintype, COMPARE_EQ);
BTEqualStrategyNumber);
if (!OidIsValid(op)) if (!OidIsValid(op))
elog(ERROR, "missing operator %d(%u,%u) in opfamily %u", elog(ERROR, "missing equality operator for (%u,%u) in opfamily %u",
BTEqualStrategyNumber, opcintype, opcintype, opfamily); opcintype, opcintype, opfamily);
/* /*
* If we find the same column with the same equality semantics * If we find the same column with the same equality semantics
@ -920,15 +918,10 @@ is_usable_unique_index(Relation indexRel)
/* /*
* Must be unique, valid, immediate, non-partial, and be defined over * Must be unique, valid, immediate, non-partial, and be defined over
* plain user columns (not expressions). We also require it to be a * plain user columns (not expressions).
* btree. Even if we had any other unique index kinds, we'd not know how
* to identify the corresponding equality operator, nor could we be sure
* that the planner could implement the required FULL JOIN with non-btree
* operators.
*/ */
if (indexStruct->indisunique && if (indexStruct->indisunique &&
indexStruct->indimmediate && indexStruct->indimmediate &&
indexRel->rd_rel->relam == BTREE_AM_OID &&
indexStruct->indisvalid && indexStruct->indisvalid &&
RelationGetIndexPredicate(indexRel) == NIL && RelationGetIndexPredicate(indexRel) == NIL &&
indexStruct->indnatts > 0) indexStruct->indnatts > 0)