Re: BUG #1643: dbf2pg broken + quick fix

From: Boris van Schooten <schooten(at)cs(dot)utwente(dot)nl>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #1643: dbf2pg broken + quick fix
Date: 2005-05-06 08:58:12
Message-ID: Pine.SOL.4.33.0505051536320.25790-100000@demeter
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

I don't see this as half baked error checking, but rather, as error
reporting during a "half baked conversion". I prefer to know when the
converter encounters something it doesn't understand, rather than having
it silently enter nulls into the db.

It was the original error reporting that helped me find this bug, I
suppose.

Kind regards,
Boris van Schooten

On Thu, 5 May 2005, Tom Lane wrote:

> Boris van Schooten <schooten(at)cs(dot)utwente(dot)nl> writes:
> > Looks good to me. I'd prefer to have a warning message (if (verbose)
> > fprintf stderr) for each of the exceptional conditions though. I'm even
> > so paranoid I prefer to have the verbose switch on by default.
>
> Don't really see the need for it. What we are doing here is trusting to
> the backend to error-check the input, rather than making a half-baked
> attempt to do error checking in dbf2pg.
>
> regards, tom lane
>

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Michael Fuhr 2005-05-06 12:58:45 Re: BUG #1647: shows version 7.1, doesnt create tablespaces etc.
Previous Message Bruce Momjian 2005-05-05 16:37:40 Re: BUG #1545: LIBPQ Windows Version not calling WSACleanup for