Re: [HACKERS] NULLs in array_cat vs array || array

2011-02-15 Thread Thom Brown
On 15 February 2011 21:47, Tom Lane wrote: > Also, so far as I can see array_cat *is* ||, so I'm not sure what > discrepancy in behavior you're on about. You've confused me now. I had a case where I replaced || with , and surrounded it with array_cat, and the result differed, and now I can't rec

Re: [HACKERS] NULLs in array_cat vs array || array

2011-02-15 Thread Thom Brown
On 15 February 2011 21:46, Cédric Villemain wrote: > 2011/2/15 Thom Brown : >> Hi all, >> >> I assumed array_cat would behave similarly to array || array, but it >> appears not when it comes to NULLs.  Shouldn't these have identical >> functionality?  The attached patch makes it so, although it wo

Re: [HACKERS] NULLs in array_cat vs array || array

2011-02-15 Thread Tom Lane
Thom Brown writes: > I assumed array_cat would behave similarly to array || array, but it > appears not when it comes to NULLs. Shouldn't these have identical > functionality? The attached patch makes it so, although it would > break existing code. That patch is the hard way: the right change w

Re: [HACKERS] NULLs in array_cat vs array || array

2011-02-15 Thread Cédric Villemain
2011/2/15 Thom Brown : > Hi all, > > I assumed array_cat would behave similarly to array || array, but it > appears not when it comes to NULLs.  Shouldn't these have identical > functionality?  The attached patch makes it so, although it would > break existing code. There is bugreport and todo ent