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
This commit is contained in:
Peter Geoghegan 2024-04-07 22:13:28 -04:00
parent af7e90a277
commit 3b08133cd1
1 changed files with 0 additions and 4 deletions

View File

@ -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));
}
}