Log In - Home Page

Patch: Shave a few instructions from child-process startup sequence

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-11
Topic Performance
Patch Status Returned with Feedback
Author Gurjeet Singh
Reviewers Peter Eisentraut
Committer Nobody
Close Date 2014-01-23
Patch by singh.gurjeet on 2013-11-05 07:37:26 AM: Initial version.
Review by singh.gurjeet on 2013-11-05 07:39:50 AM: Review by Amit Kapila.
Comment by singh.gurjeet on 2013-11-05 07:40:56 AM: Tom Lane's reservations.
Comment by singh.gurjeet on 2013-11-05 07:41:58 AM: Gurjeet Singh's follow-up.
Comment by singh.gurjeet on 2013-11-05 07:43:37 AM: Amit Kapila's support for the arguments being made in favor of the patch.
Comment by singh.gurjeet on 2013-11-05 07:46:10 AM: Tom Lane's reservations about the benefit of the patch.
Comment by singh.gurjeet on 2013-11-05 07:49:47 AM: Gurjeet Singh's counter-argument that the patch helps in backend startup, as opposed to Postmaster shutdown (perhaps patch's location was misunderstood by others).
Comment by petere on 2013-11-07 10:04:53 PM: build OK: http://pgci.eisentraut.org/jenkins/job/postgresql_commitfest_world/173/
Patch by singh.gurjeet on 2013-11-23 04:35:13 AM: Patch that adds aseertions to make sure the assumptions hold true. On Peter E's suggestion.
Comment by petere on 2013-11-27 02:42:21 AM: build unstable: http://pgci.eisentraut.org/jenkins/job/postgresql_commitfest_world/243/

Add Comment

Please log in to comment on this patch.