MonetDB: SciQL-2 - handle non-rectangular array tiles
Stefan.Manegold at cwi.nl
Thu Aug 15 07:53:45 CEST 2013
> Changeset: 58e31783ae0f for MonetDB
> URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=58e31783ae0f
> Modified Files:
> Branch: SciQL-2
> Log Message:
> handle non-rectangular array tiles
> - accept either interval tile range, or point tile ragnes, no mix of them
> - multiple tile ranges are appended to e_column->f, thus e_column->f list can
> have more than 3 items.
> - to distinguish a point range from an interval range: a point range only has
> one expr., while an interval range has three
> - save the values for the error message, since the lists are freed in
Nice catch ;-) --- Thanks for fixing! ... and sorry for overlooking it myself ;-)
> - disable the check on (max - min + 1 = size), since it only holds for
> stepsize = 1.
> maybe we should just remove this check, since we don't have step size in
> tiling aggr impl. functions, do we?
IMHO, our current tiled aggregation implementation inherently assumes (and thus requires) that all dimensions of the given array have step-size == 1. Since ARRAYtiles*() does not get the step sizes of the given array as parameters, the check (max - min + 1 = size) was/is IMHO the only way to check/ensure that we are indeed working on an array where all dimensions have step-size == 1.
> now we can get rid of the workarounds
> more tests needed...
Indeed --- I'll resume me test tonight or tomorrow ...
| Stefan.Manegold at CWI.nl | DB Architectures (DA) |
| www.CWI.nl/~manegold/ | Science Park 123 (L321) |
| +31 (0)20 592-4212 | 1098 XG Amsterdam (NL) |
More information about the developers-list