Re: Tsearch2 update for Win32. - Mailing list pgsql-patches

From Tom Lane
Subject Re: Tsearch2 update for Win32.
Date
Msg-id 20222.1094654950@sss.pgh.pa.us
Whole thread Raw
In response to Re: Tsearch2 update for Win32.  ("Magnus Hagander" <mha@sollentuna.net>)
Responses Re: Tsearch2 update for Win32.  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-patches
"Magnus Hagander" <mha@sollentuna.net> writes:
>> Oh.  Hmm ... wouldn't it be easier and safer to launch "psql <script"
>> as a subprocess?

> I'd say no. Executing processes from the installer environment can be a
> headache (we've had enough problems with initdb already..). And you have
> to go through all the weirdness with the commandline quoting etc on
> win32... It's a lot easier to execute a SQL script line-by-line. Which
> also lets us trap the errors easier etc.

I was about to mention trapping errors as being a likely weak spot of
custom-script-execution code.  And have you duplicated psql's rather
extensive logic for determining where SQL commands end?  I'll get a bit
annoyed if you come back later and tell us not to use dollar quoting
in contrib scripts, or something like that.

However, the COPY->INSERT change alone is below my threshold of pain,
so I won't complain too hard.  I just wanted to be sure you'd really
thought about the implications of rolling your own script executor.

            regards, tom lane

pgsql-patches by date:

Previous
From: "Magnus Hagander"
Date:
Subject: Re: Tsearch2 update for Win32.
Next
From: Fabien COELHO
Date:
Subject: Re: Tsearch2 update for Win32.