forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit8b304b8
committed
Remove replacement selection sort.
At the time replacement_sort_tuples was introduced, there were stillcases where replacement selection sort noticeably outperformed usingquicksort even for the first run. However, those cases seem to haveevaporated as a result of further improvements made since that time(and perhaps also advances in CPU technology). So remove replacementselection and the controlling GUC entirely. This makes tuplesort.cnoticeably simpler and probably paves the way for furtheroptimizations someone might want to do later.Peter Geoghegan, with review and testing by Tomas Vondra and me.Discussion:https://postgr.es/m/CAH2-WzmmNjG_K0R9nqYwMq3zjyJJK+hCbiZYNGhAy-Zyjs64GQ@mail.gmail.com1 parentd2773f9 commit8b304b8
File tree
9 files changed
+52
-448
lines changed- doc/src/sgml
- src
- backend/utils
- init
- misc
- sort
- include
- test/regress
- expected
- sql
9 files changed
+52
-448
lines changedLines changed: 0 additions & 39 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1505 | 1505 |
| |
1506 | 1506 |
| |
1507 | 1507 |
| |
1508 |
| - | |
1509 |
| - | |
1510 |
| - | |
1511 |
| - | |
1512 |
| - | |
1513 |
| - | |
1514 |
| - | |
1515 |
| - | |
1516 |
| - | |
1517 |
| - | |
1518 |
| - | |
1519 |
| - | |
1520 |
| - | |
1521 |
| - | |
1522 |
| - | |
1523 |
| - | |
1524 |
| - | |
1525 |
| - | |
1526 |
| - | |
1527 |
| - | |
1528 |
| - | |
1529 |
| - | |
1530 |
| - | |
1531 |
| - | |
1532 |
| - | |
1533 |
| - | |
1534 |
| - | |
1535 |
| - | |
1536 |
| - | |
1537 |
| - | |
1538 |
| - | |
1539 |
| - | |
1540 |
| - | |
1541 |
| - | |
1542 |
| - | |
1543 |
| - | |
1544 |
| - | |
1545 |
| - | |
1546 |
| - | |
1547 | 1508 |
| |
1548 | 1509 |
| |
1549 | 1510 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
5140 | 5140 |
| |
5141 | 5141 |
| |
5142 | 5142 |
| |
5143 |
| - | |
| 5143 | + | |
5144 | 5144 |
| |
5145 | 5145 |
| |
5146 | 5146 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
112 | 112 |
| |
113 | 113 |
| |
114 | 114 |
| |
115 |
| - | |
116 | 115 |
| |
117 | 116 |
| |
118 | 117 |
| |
|
Lines changed: 0 additions & 10 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1933 | 1933 |
| |
1934 | 1934 |
| |
1935 | 1935 |
| |
1936 |
| - | |
1937 |
| - | |
1938 |
| - | |
1939 |
| - | |
1940 |
| - | |
1941 |
| - | |
1942 |
| - | |
1943 |
| - | |
1944 |
| - | |
1945 |
| - | |
1946 | 1936 |
| |
1947 | 1937 |
| |
1948 | 1938 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
121 | 121 |
| |
122 | 122 |
| |
123 | 123 |
| |
124 |
| - | |
125 | 124 |
| |
126 | 125 |
| |
127 | 126 |
| |
|
0 commit comments
Comments
(0)