diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2024-12-10 12:30:42 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2024-12-10 12:30:42 +0100 |
| commit | 321c287351f707c38c1fa94c1ac89bcc3134ed59 (patch) | |
| tree | 00bd04b5df948dd05769c444b094a7a20206464f | |
| parent | d37e856410d0856cb851e11b2e0191edf6cde527 (diff) | |
| download | postgresql-321c287351f707c38c1fa94c1ac89bcc3134ed59.tar.gz postgresql-321c287351f707c38c1fa94c1ac89bcc3134ed59.zip | |
Improve internal logical replication error for missing equality strategy
This "shouldn't happen", except right now it can with a temporal gist
index (to be fixed soon), because of missing gist support in
get_equal_strategy_number(). But right now, the error is not caught
right away, but instead you get the subsequent error about a "missing
operator 0". This makes the error more accurate.
Author: Paul Jungwirth <pj@illuminatedcomputing.com>
Discussion: https://www.postgresql.org/message-id/flat/CA+renyUApHgSZF9-nd-a0+OPGharLQLO=mDHcY4_qQ0+noCUVg@mail.gmail.com
| -rw-r--r-- | src/backend/executor/execReplication.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/executor/execReplication.c b/src/backend/executor/execReplication.c index 368a40337aa..c8b79f42c24 100644 --- a/src/backend/executor/execReplication.c +++ b/src/backend/executor/execReplication.c @@ -134,6 +134,8 @@ build_replindex_scan_key(ScanKey skey, Relation rel, Relation idxrel, optype = get_opclass_input_type(opclass->values[index_attoff]); opfamily = get_opclass_family(opclass->values[index_attoff]); eq_strategy = get_equal_strategy_number(opclass->values[index_attoff]); + if (!eq_strategy) + elog(ERROR, "missing equal strategy for opclass %u", opclass->values[index_attoff]); operator = get_opfamily_member(opfamily, optype, optype, |