Re: insert performance for win32

Lists: pgsql-hackers
From: "Merlin Moncure" <merlin(dot)moncure(at)rcsonline(dot)com>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: insert performance for win32
Date: 2005-11-04 18:26:52
Message-ID: 6EE64EF3AB31D5448D0007DD34EEB3417DD7D6@Herge.rcsinc.local
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


> void
> setup_cancel_handler(void)
> {
> + static bool done = false;
> +
> + if (!done)
> SetConsoleCtrlHandler(consoleHandler, TRUE);
> + done = true;
> }
>

That works, I tried ctrl-c various ways including from within \i copy.
Problem solved!

Merlin


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Merlin Moncure" <merlin(dot)moncure(at)rcsonline(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: insert performance for win32
Date: 2005-11-04 18:37:32
Message-ID: 16427.1131129452@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

"Merlin Moncure" <merlin(dot)moncure(at)rcsonline(dot)com> writes:
> That works, I tried ctrl-c various ways including from within \i copy.
> Problem solved!

Good. I've applied the patch in both HEAD and 8.0 branches.

Since we're very nearly ready to wrap 8.1, would someone with access to
a Windows machine please double-check that CVS tip still works?

regards, tom lane


From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Merlin Moncure <merlin(dot)moncure(at)rcsonline(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: insert performance for win32
Date: 2005-11-04 21:03:59
Message-ID: 436BCCBF.9030704@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:

>"Merlin Moncure" <merlin(dot)moncure(at)rcsonline(dot)com> writes:
>
>
>>That works, I tried ctrl-c various ways including from within \i copy.
>>Problem solved!
>>
>>
>
>Good. I've applied the patch in both HEAD and 8.0 branches.
>
>Since we're very nearly ready to wrap 8.1, would someone with access to
>a Windows machine please double-check that CVS tip still works?
>
>
>
>

Worked for me. See buildfarm. Or are there more tests you want run?

cheers

andrew