On Wed, May 25, 2016 at 9:44 PM, Michael Paquier <michael.paqu...@gmail.com>
> On Wed, May 25, 2016 at 12:11 AM, Amit Kapila <amit.kapil...@gmail.com>
> >
> > Okay, attached patch just does that and I have verified that it allows
> > start multiple services in windows.  In off list discussion with
Robert, he
> > suggested not to complicate the patch by retrying for fixed number of
> > as there is no proof that ERROR_ACCESS_DENIED can occur due to any other
> > reason in this code path.  This patch is based on Kyotaro san's patch
> > upthread with just minor changes in comments and indentation.
> Thanks for catching Robert and getting confirmation on the matter. In
> the same line of thoughts, I think as well that it is definitely not
> worth complicating the retry logic in dsm.c, but you knew that already
> per last week's discussion.
> Regarding the patch, this looks good to me.

Thanks for reviewing the patch.  I have added the entry for this patch in
next CF (https://commitfest.postgresql.org/10/636/),  feel free to mark it
as Ready for committer if you think patch is good.

With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

Reply via email to