On Thu, Apr 25, 2013 at 07:33:43AM -0600, Marcus Sorensen wrote: > 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. >
Done! > > On Thu, Apr 25, 2013 at 7:32 AM, Marcus Sorensen <[email protected]>wrote: > > > yes > > > > > > On Thu, Apr 25, 2013 at 7:31 AM, Chip Childers > > <[email protected]>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 < > >> [email protected]>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 > >> > > > >> > > > >
