diff options
| author | Peter Geoghegan <pg@bowt.ie> | 2020-01-05 17:42:13 -0800 |
|---|---|---|
| committer | Peter Geoghegan <pg@bowt.ie> | 2020-01-05 17:42:13 -0800 |
| commit | fc3100112395485f1c65848b273b3235de4aad07 (patch) | |
| tree | 5b9e8230bbae060706f517a2f52496dd3d6cdbcc | |
| parent | 8c081a2f4e8eec11747e709c1ea4c3485ef129dd (diff) | |
| download | postgresql-fc3100112395485f1c65848b273b3235de4aad07.tar.gz postgresql-fc3100112395485f1c65848b273b3235de4aad07.zip | |
Remove redundant incomplete split assertion.
The fastpath insert optimization's incomplete split flag Assert() is
redundant. We'll reach the more general Assert() within
_bt_findinsertloc() in all cases. (Besides, Assert()'ing that the
rightmost page doesn't have the flag set never made much sense.)
| -rw-r--r-- | src/backend/access/nbtree/nbtinsert.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/backend/access/nbtree/nbtinsert.c b/src/backend/access/nbtree/nbtinsert.c index 144d339e8d4..7ddba3ff9ff 100644 --- a/src/backend/access/nbtree/nbtinsert.c +++ b/src/backend/access/nbtree/nbtinsert.c @@ -179,11 +179,6 @@ top: PageGetMaxOffsetNumber(page) >= P_FIRSTDATAKEY(lpageop) && _bt_compare(rel, itup_key, page, P_FIRSTDATAKEY(lpageop)) > 0) { - /* - * The right-most block should never have an incomplete split. - * But be paranoid and check for it anyway. - */ - Assert(!P_INCOMPLETE_SPLIT(lpageop)); fastpath = true; } else |