Marco Nenciarini marco.nenciar...@2ndquadrant.it writes:
Patch v3 attached.
I'm looking at this patch now. The restriction of array_remove to
one-dimensional arrays seems a bit annoying. I see the difficulty:
if the input is multi-dimensional then removing some elements could
lead to a
On Wed, Jul 11, 2012 at 9:54 AM, Tom Lane t...@sss.pgh.pa.us wrote:
Marco Nenciarini marco.nenciar...@2ndquadrant.it writes:
Patch v3 attached.
I'm looking at this patch now. The restriction of array_remove to
one-dimensional arrays seems a bit annoying. I see the difficulty:
if the input
Alex Hunsaker bada...@gmail.com writes:
On Wed, Jul 11, 2012 at 9:54 AM, Tom Lane t...@sss.pgh.pa.us wrote:
I'm looking at this patch now. The restriction of array_remove to
one-dimensional arrays seems a bit annoying. I see the difficulty:
if the input is multi-dimensional then removing
On Jul 11, 2012, at 11:53 AM, Alex Hunsaker bada...@gmail.com wrote:
On Wed, Jul 11, 2012 at 9:54 AM, Tom Lane t...@sss.pgh.pa.us wrote:
Marco Nenciarini marco.nenciar...@2ndquadrant.it writes:
Patch v3 attached.
I'm looking at this patch now. The restriction of array_remove to
Marco Nenciarini marco.nenciar...@2ndquadrant.it writes:
Patch v3 attached.
Applied with mostly-but-not-entirely cosmetic adjustments.
I left array_remove throwing error for multi-dimensional arrays for
the moment, because I realized that changing the dimensionality as
I suggested would
On Sat, Jun 30, 2012 at 3:28 PM, Marco Nenciarini
marco.nenciar...@2ndquadrant.it wrote:
On 30/06/2012 04:16, Alex Hunsaker wrote:
Hi, I've been reviewing this patch.
Good documentation, and regression tests. The code looked fine but I
didn't care for the code duplication between
On 30/06/2012 04:16, Alex Hunsaker wrote:
Hi, I've been reviewing this patch.
Good documentation, and regression tests. The code looked fine but I
didn't care for the code duplication between array_replace and
array_remove so I merged those into a helper function,
On Thu, Jun 14, 2012 at 4:41 AM, Marco Nenciarini
marco.nenciar...@2ndquadrant.it wrote:
Hi,
following Gabriele's email regarding our previous patch on Foreign
Key Arrays[1], I am sending a subset of that patch which includes only
two array functions which will be needed in that patch:
Hi,
following Gabriele's email regarding our previous patch on Foreign
Key Arrays[1], I am sending a subset of that patch which includes only
two array functions which will be needed in that patch: array_remove
(limited to single-dimensional arrays) and array_replace.
The patch includes