On Sun, 2014-07-06 at 21:11 -0700, Jeff Davis wrote:
> On Wed, 2014-04-16 at 12:50 +0100, Nicholas White wrote:
> > Thanks for the detailed feedback, I'm sorry it took so long to
> > incorporate it. I've attached the latest version of the patch, fixing
> > in particular:
Looking a little more:
* No tests exercise non-const offsets
* No tests for default clauses with IGNORE NULLS
* The use of bitmapsets is quite ugly. It would be nice if the API would
grow the BMS within the memory context in which it was allocated, but I
don't even see that the BMS is necessary. Why not just allocate a
fixed-size array of bits, and forget the BMS?
* Is there a reason you're leaving out first_value/last_value/nth_value?
I think they could be supported without a lot of extra work.
Regards,Jeff Davis
