Sorry, I should have mentioned in my first response that I had just removed the attachment from the review request. That would have clarified things.
On Thu, Apr 25, 2013 at 7:32 AM, Marcus Sorensen <shadow...@gmail.com>wrote: > yes > > > On Thu, Apr 25, 2013 at 7:31 AM, Chip Childers > <chip.child...@sungard.com>wrote: > >> On Thu, Apr 25, 2013 at 07:29:55AM -0600, Marcus Sorensen wrote: >> > Sorry, I attached the patch rather than updating the review diff >> because it >> > was just a quick idea and I wanted to get some feedback on it. After >> > working a bit more, I came up with a better solution based on that idea >> and >> > updated the diff with it. >> >> OK, so I should apply what I see here: >> >> https://reviews.apache.org/r/10766/diff/raw/ >> >> Just want to confirm... >> >> > >> > >> > On Thu, Apr 25, 2013 at 7:25 AM, Chip Childers < >> chip.child...@sungard.com>wrote: >> > >> > > On Wed, Apr 24, 2013 at 03:24:20PM -0600, Marcus Sorensen wrote: >> > > > https://reviews.apache.org/r/10766/ >> > > >> > > I'm unclear about the current state of this review request. There is >> > > the actual patch for the review [1] and an attached patch [2]. >> > > >> > > Which should be applied? >> > > >> > > -chip >> > > >> > > [1] https://reviews.apache.org/r/10766/diff/raw/ >> > > [2] >> > > >> https://reviews.apache.org/media/uploaded/files/2013/04/25/patch-fix-ha-primary-only >> > > >> > >
