Removing replacement selection sort (replacement_sort_tuples GUC) from tuplesort.c

ID 1278

Title Removing replacement selection sort (replacement_sort_tuples GUC) from tuplesort.c

Topic Refactoring

Created 2017-08-31 18:01:50

Last modified 2017-09-29 14:29:26 (7 years, 1 month ago)

Latest email 2017-09-29 16:20:55 (7 years, 1 month ago)

Status

2017-09: Committed

Target version

Authors Peter Geoghegan (pgeoghegan)

Reviewers Become reviewer

Committer Robert Haas (rhaas)

Links CFbot results (CirrusCI) CFbot GitHub

Checkout latest CFbot patchset

Go to your local checkout of the PostgreSQL repository and run:

git remote add commitfest https://github.com/postgresql-cfbot/postgresql.git
git fetch commitfest cf/1278
git checkout commitfest/cf/1278

Emails

First at 2017-07-14 22:20:39 by Peter Geoghegan <pg at bowt.ie>
Latest at 2017-09-29 16:20:55 by Peter Geoghegan <pg at bowt.ie>
Latest attachment (sort-benchmarks.ods) at 2017-09-15 13:34:34 from Tomas Vondra <tomas.vondra at 2ndquadrant.com>

    Attachment (sort-benchmarks.ods) at 2017-09-15 13:34:34 from Tomas Vondra <tomas.vondra at 2ndquadrant.com> (Patch: No)
    Attachment (0001-Remove-replacement-selection-sort.patch) at 2017-09-11 19:31:01 from Peter Geoghegan <pg at bowt.ie> (Patch: Yes)
    Attachment (e5-2620-v4.ods) at 2017-09-11 15:17:09 from Tomas Vondra <tomas.vondra at 2ndquadrant.com> (Patch: No)
    Attachment (sort-bench.sh) at 2017-09-11 00:07:36 from Tomas Vondra <tomas.vondra at 2ndquadrant.com> (Patch: No)
    Attachment (0001-Remove-replacement-selection-sort.patch) at 2017-08-31 18:00:54 from Peter Geoghegan <pg at bowt.ie> (Patch: Yes)

When	Who	Mail	Annotation
2017-09-15 20:42:56	Peter Geoghegan (pgeoghegan)	From Peter Geoghegan <pg@bowt.ie> at 2017-09-15 20:40:41	Benchmark from Tomas shows conclusively that removing replacement selection sort is a performance win

History

When	Who	What
2017-09-29 14:29:26	Robert Haas (rhaas)	Closed in commitfest 2017-09 with status: Committed
2017-09-29 14:29:26	Robert Haas (rhaas)	Changed committer to rhaas
2017-09-15 20:42:56	Peter Geoghegan (pgeoghegan)	Added annotation "Benchmark from Tomas shows conclusively that removing replacement selection sort is a performance win" to CAH2-Wz=SkAZNLcdzR1rn+_4mTqSy=_st+k-O2wrZHcGVo1eHvw@mail.gmail.com
2017-09-15 20:42:34	Peter Geoghegan (pgeoghegan)	Deleted annotation "Benchmark from Tomas shows conclusively that removing replacement selection sort is a performance win" from CAH2-WznFOrDV2Ht3e1bAM64anRUYK6G8ao7ABbLCHJBWLY+asQ@mail.gmail.com
2017-09-15 18:43:12	Peter Geoghegan (pgeoghegan)	Added annotation "Benchmark from Tomas shows conclusively that removing replacement selection sort is a performance win" to CAH2-WznFOrDV2Ht3e1bAM64anRUYK6G8ao7ABbLCHJBWLY+asQ@mail.gmail.com
2017-08-31 18:02:01	Peter Geoghegan (pgeoghegan)	Changed authors to Peter Geoghegan (pgeoghegan)
2017-08-31 18:01:51	Peter Geoghegan (pgeoghegan)	Attached mail thread CAH2-WzmmNjG_K0R9nqYwMq3zjyJJK+hCbiZYNGhAy-Zyjs64GQ@mail.gmail.com
2017-08-31 18:01:50	Peter Geoghegan (pgeoghegan)	Created patch record