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 Tom Lane
Manfred Koizar <[EMAIL PROTECTED]> writes: > Should I send a new patch with only the regression tests? Yeah, if you think they are worth adding. regards, tom lane ---(end of broadcast)--- TIP 9: the planner will ignore your

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

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: > > Mak

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 C

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

2003-10-08 Thread Tom Lane
Manfred Koizar <[EMAIL PROTECTED]> writes: > Here is a patch that does this and adds a few regression tests. Uh, I did that already ... for 7.4 at least. regards, tom lane ---(end of broadcast)--- TIP 9: the planner will ign

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 m