aboutsummaryrefslogtreecommitdiff
path: root/src/backend/tcop/postgres.c
diff options
context:
space:
mode:
authorPeter Geoghegan <pg@bowt.ie>2020-03-10 14:15:41 -0700
committerPeter Geoghegan <pg@bowt.ie>2020-03-10 14:15:41 -0700
commitd1e241c22602a5369b54e29b7430553445f6a52b (patch)
tree5e071dc20aee38a28f18db15d1e734c098add02f /src/backend/tcop/postgres.c
parentcacef172237fd3426b578f81b7414c0de56cbaaf (diff)
downloadpostgresql-d1e241c22602a5369b54e29b7430553445f6a52b.tar.gz
postgresql-d1e241c22602a5369b54e29b7430553445f6a52b.zip
nbtree: Demote minus infinity "can't happen" error.
Only a very basic logic bug in a _bt_insertonpg() caller could lead to a violation of this invariant. Besides, any newitemoff used for an internal page is sanitized using other "can't happen" errors in _bt_getstackbuf() or its callers, before _bt_insertonpg() even gets called. Also, move the error/assertion from the insert-without-split path of _bt_insertonpg() to the top of the same function. There is no reason why this invariant only applies to insertions that happen to not result in a page split; cover every insertion. The assertion naturally belongs next to the existing generic assertions that document relatively high-level invariants for the item being inserted.
Diffstat (limited to 'src/backend/tcop/postgres.c')
0 files changed, 0 insertions, 0 deletions