[PATCH 3 of 3] worker: handle worker failures more aggressively
Isaac Jurado
diptongo at gmail.com
Thu Feb 21 15:58:48 CST 2013
Replying Bryan O'Sullivan:
> On Thu, Feb 21, 2013 at 12:22 AM, Isaac Jurado <diptongo at gmail.com> wrote:
>
>> That's my point. Working with processes on Windows is going to be
>> different enough from the other platforms, right?
>
> Okay, I'm confused. Are you looking for me to do something, or just
> passing the time?
Last time I reply on this, I promise. Let me summarize what, from my
point of view, has happened.
- You submitted a patch that uses a monitor thread on the parent
process in order to detect a children failure early and finish the
rest of the children.
- I suggested to leave that task to the children processes by putting
all of them in the same process group and "broadcasting" a signal on
failure. Which could circumvent the need for threading the parent
process.
- You replied that process groups are Unix only and you wanted to
maximize code portability.
- I was confused, since os.fork() and atomic writes to pipes are also
Unix only (POSIX in particular). So I tried to ask for a better
explanation, as politely as possible. Specially since mailing list
discussions tend to heat up very quickly and, between you and me,
you are the authority. I don't want any trouble.
- After a pair of replies, you told me, more or less, that Windows is
a whole different story. Statement that I found very contradictory
from the first argument rejecting the use of process groups.
Therefore, when, in theroy, I was trying to understand the resasonings behind some
design decisions, in practice I was unwillingly passing the time.
As I said, there's no point in taking this discussion further. Whatever
you write will be fine, even if apparently contradictory and if I have
any complaint, I'll send a patch.
Sorry for the confusion. Best regards.
--
Isaac Jurado
"The noblest pleasure is the joy of understanding."
Leonardo da Vinci
More information about the Mercurial-devel
mailing list