Re: [PATCHES] [HACKERS] Cannot dump/restore text value \N

2003-10-09 Thread Bruce Momjian
Tom Lane wrote: Manfred Koizar [EMAIL PROTECTED] writes: Should I send a new patch with only the regression tests? Yeah, if you think they are worth adding. Regression part of NULL patch applied. -- Bruce Momjian| http://candle.pha.pa.us [EMAIL PROTECTED]

Re: [PATCHES] [HACKERS] Cannot dump/restore text value \N

2003-10-08 Thread Manfred Koizar
On Sun, 05 Oct 2003 19:12:50 -0400, Tom Lane [EMAIL PROTECTED] wrote: it seems we have to compare the null representation string to the pre-debackslashing input. Here is a patch that does this and adds a few regression tests. (This is probably fairly easy to make happen in CVS tip, but it might

Re: [PATCHES] [HACKERS] Cannot dump/restore text value \N

2003-10-08 Thread Manfred Koizar
On Wed, 08 Oct 2003 11:31:30 +0200, I wrote: There haven't been too much changes in this area between 7.3 and 7.4. Here is the patch for 7.3.4 ... Bruce, I noticed that the original patch submission didn't contain anything useful as a cvs log message: Make COPY FROM a bit more compatible with

Re: [PATCHES] [HACKERS] Cannot dump/restore text value \N

2003-10-08 Thread Bruce Momjian
Manfred Koizar wrote: On Wed, 08 Oct 2003 11:31:30 +0200, I wrote: There haven't been too much changes in this area between 7.3 and 7.4. Here is the patch for 7.3.4 ... Bruce, I noticed that the original patch submission didn't contain anything useful as a cvs log message: Make COPY

Re: [PATCHES] [HACKERS] Cannot dump/restore text value \N

2003-10-08 Thread Manfred Koizar
On Wed, 8 Oct 2003 11:33:24 -0400 (EDT), Bruce Momjian [EMAIL PROTECTED] wrote: Can someone explain what was broken? COPY FROM removed backslashes before comparing the input to the external null representation. (It had a hard-wired special code path that allowed \N to be recognized.) The text