Proposed patch for contrib/cube
Hi,
I use the cube datatype a fair bit, and one thing I have always wanted
is the ability to do this:
pg=# select cube_from_arrays('{1,2,3}'::float[], '{3,5,6}'::float[]);
cube_from_arrays
---------------------
(1, 2, 3),(3, 5, 6)
(1 row)
That is - build a cube by specifying 2 arrays, one for the UR
coordinate, one for LL.
I hope people find this useful, and if so, we can add it to contrib/cube.
Source is attached.
Thanks,
Joshua Reich
(jdigittl on #postgresql)
Attachments:
Joshua Reich <josh@root.net> writes:
... build a cube by specifying 2 arrays, one for the UR
coordinate, one for LL.
I hope people find this useful, and if so, we can add it to contrib/cube.
Seems useful, but it needs work: it will fail on toasted arrays or
arrays containing nulls. I'd suggest converting it to V1 call protocol
too, as the V1 GETARG macros are the easiest answer to the toasting
problem. Null array elements are a new issue in CVS HEAD, you'd need
to test against HEAD for that.
As a matter of style, would it be better to error out if the arrays
are not the same length?
regards, tom lane
Tom: Thanks for the out-of-band posting to the documentation. I think
the new version (attached) addresses your issues.
What is the general process for submitting patches? Is there a URL
someone can point me towards to learn more?
Thanks,
Josh Reich
Tom Lane wrote:
Show quoted text
Joshua Reich <josh@root.net> writes:
... build a cube by specifying 2 arrays, one for the UR
coordinate, one for LL.
I hope people find this useful, and if so, we can add it to contrib/cube.Seems useful, but it needs work: it will fail on toasted arrays or
arrays containing nulls. I'd suggest converting it to V1 call protocol
too, as the V1 GETARG macros are the easiest answer to the toasting
problem. Null array elements are a new issue in CVS HEAD, you'd need
to test against HEAD for that.As a matter of style, would it be better to error out if the arrays
are not the same length?regards, tom lane
---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?
