diff options
| author | Peter Geoghegan <pg@bowt.ie> | 2020-04-15 15:47:26 -0700 |
|---|---|---|
| committer | Peter Geoghegan <pg@bowt.ie> | 2020-04-15 15:47:26 -0700 |
| commit | f0ca378d4c139eda99ef14998115c1674dac3fc5 (patch) | |
| tree | e198a472cadb98e7e7a1448c7473a7ee18914025 /src | |
| parent | 8f4ee44bcdbcd437f860f2d048125a118335b088 (diff) | |
| download | postgresql-f0ca378d4c139eda99ef14998115c1674dac3fc5.tar.gz postgresql-f0ca378d4c139eda99ef14998115c1674dac3fc5.zip | |
Slightly simplify nbtree split point choice loop.
Spotted during post-commit review of the nbtree deduplication commit
(commit 0d861bbb).
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/nbtree/nbtsplitloc.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/backend/access/nbtree/nbtsplitloc.c b/src/backend/access/nbtree/nbtsplitloc.c index 5f0d0be3c25..c850cd807cf 100644 --- a/src/backend/access/nbtree/nbtsplitloc.c +++ b/src/backend/access/nbtree/nbtsplitloc.c @@ -820,18 +820,14 @@ _bt_bestsplitloc(FindSplitData *state, int perfectpenalty, penalty = _bt_split_penalty(state, state->splits + i); - if (penalty <= perfectpenalty) - { - bestpenalty = penalty; - lowsplit = i; - break; - } - if (penalty < bestpenalty) { bestpenalty = penalty; lowsplit = i; } + + if (penalty <= perfectpenalty) + break; } final = &state->splits[lowsplit]; |