Re: Review of Refactoring code for sync node detection - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Review of Refactoring code for sync node detection
Date
Msg-id CAB7nPqTdPX3A-k7AYSX4YTqONPe2Swi_gkOds-4U5hczdYRFsw@mail.gmail.com
Whole thread Raw
In response to Re: Review of Refactoring code for sync node detection  (Jim Nasby <Jim.Nasby@BlueTreble.com>)
Responses Re: Review of Refactoring code for sync node detection
List pgsql-hackers
On Fri, Oct 31, 2014 at 6:59 AM, Jim Nasby <Jim.Nasby@bluetreble.com> wrote:
If we stick with this version I'd argue it makes more sense to just stick the sync_node = and priority = statements into the if block and ditch the continue. </nitpick>
Let's go with the cleaner version then, I'd prefer code that can be read easily for this code path. Switching back is not much complicated either.
--
Michael
Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: TAP test breakage on MacOS X
Next
From: Tom Lane
Date:
Subject: Re: TAP test breakage on MacOS X