Log In - Home Page

Patch: Print sort ordering options in EXPLAIN

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2014-12
Topic Server Features
Patch Status Committed
Author Marius Timmer, Lukas Kreft, Arne Scheffer
Reviewers Mike Blackwell
Committer Tom Lane
Close Date 2015-01-16
Comments
Patch by mtimm_01 on 2014-11-25 11:39:52 AM: Initial version.
Review by maiku41 on 2014-12-15 04:52:15 PM: Patch applies. Make test fails with missing collation clause. Minor formatting (space/tab) issues.
Patch by mtimm_01 on 2014-12-16 10:01:22 AM: Spaces replaced with tabs. "Make test" runs successfully.
Review by heikki on 2014-12-17 03:35:53 PM: I don't like this output
Comment by arnold_s on 2015-01-06 07:03:31 PM: WIP
Patch by mtimm_01 on 2015-01-07 04:23:45 PM: First patch containing the changes Heikki's suggested
Review by maiku41 on 2015-01-08 03:43:22 PM: Review of v6 patch. Issues with memory handling. Needs doc and test updates.
Patch by mtimm_01 on 2015-01-13 04:09:34 PM: Patch using StringInfo. Doc and test updates will follow.
Comment by maiku41 on 2015-01-14 04:31:46 PM: Looks good other than quoting issue with collation name.
Waiting for doc and test update.
Patch by mtimm_01 on 2015-01-15 03:09:05 PM: Version 8, fixing remaining issues (e.g. quoting), adding docs and tests as requested/agreed.
Review by maiku41 on 2015-01-15 04:48:35 PM: Remaining issues addressed. Looks good to me. Marking as ready for committer.

Add Comment

Please log in to comment on this patch.