From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Marko Tiikkaja <marko(at)joh(dot)to> |
Cc: | Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: count_nulls(VARIADIC "any") |
Date: | 2016-01-26 11:20:11 |
Message-ID: | CAFj8pRDg2YKY++cPNYDRdUgt4ppJ7d=WFJ_uPHYw5DbxzEPK=g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2016-01-26 11:42 GMT+01:00 Marko Tiikkaja <marko(at)joh(dot)to>:
> On 25/01/16 19:57, Pavel Stehule wrote:
>
>> Marco is a author of this patch, so - Marco, please, send final version of
>> this patch
>>
>
> I don't really care about the tests. Can we not use the v5 patch already
> in the thread? As far as I could tell there were no reviewer's comments on
> it anymore.
>
It was not my request, but I counted Jim as second reviewer.
So, I'll return back original regress tests. If I remember well, there are
no any other objection, so I'll mark this version as ready for commiter.
1. the patch is rebased against master
2. now warning or errors due compilation
3. all tests are passed
4. the code is simple without side effects and possible negative
performance impacts
6. there was not objections against the design
7. the iteration over null bitmap is used more times in our code, but this
is new special case. We don't need iterate over array elements, so we
should not to use existing array iterators.
I'll mark this patch as ready for commiter
Regards
Pavel
>
> .m
>
Attachment | Content-Type | Size |
---|---|---|
num_nulls_v6.patch | text/x-patch | 13.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | leo | 2016-01-26 12:33:13 | Re: pglogical - logical replication contrib module |
Previous Message | Sebastien Diemer | 2016-01-26 11:18:31 | Re: pglogical most basic setup for logical replication |