diff options
| author | Peter Geoghegan <pg@bowt.ie> | 2024-04-07 22:13:28 -0400 |
|---|---|---|
| committer | Peter Geoghegan <pg@bowt.ie> | 2024-04-07 22:13:28 -0400 |
| commit | 3b08133cd13c4ac985a94222280160282ba35838 (patch) | |
| tree | 60a38a0faa2cf6c7dfc9667a3a2f0149fe7c1a5b /src | |
| parent | af7e90a2778625d63add69165fb68c370509a199 (diff) | |
| download | postgresql-3b08133cd13c4ac985a94222280160282ba35838.tar.gz postgresql-3b08133cd13c4ac985a94222280160282ba35838.zip | |
Remove redundant nbtree preprocessing assertions.
One of the assertions was the subject of a false positive complaint from
Coverity, but none of the assertions added much, so get rid of them.
Reported-By: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/3000247.1712537309@sss.pgh.pa.us
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/nbtree/nbtutils.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/backend/access/nbtree/nbtutils.c b/src/backend/access/nbtree/nbtutils.c index 9b9efe062fc..2eff34c4aa6 100644 --- a/src/backend/access/nbtree/nbtutils.c +++ b/src/backend/access/nbtree/nbtutils.c @@ -2937,7 +2937,6 @@ _bt_preprocess_keys(IndexScanDesc scan) if (j != (BTEqualStrategyNumber - 1) || !(xform[j].skey->sk_flags & SK_SEARCHARRAY)) { - Assert(!array || array->scan_key == i); xform[j].skey = cur; xform[j].ikey = i; xform[j].arrayidx = arrayidx; @@ -2951,9 +2950,6 @@ _bt_preprocess_keys(IndexScanDesc scan) * scan key. _bt_compare_scankey_args expects us to * always keep arrays (and discard non-arrays). */ - Assert(j == (BTEqualStrategyNumber - 1)); - Assert(xform[j].skey->sk_flags & SK_SEARCHARRAY); - Assert(xform[j].ikey == array->scan_key); Assert(!(cur->sk_flags & SK_SEARCHARRAY)); } } |