On Wed, Feb 24, 2016 at 12:30 AM, Tomas Vondra <tomas.von...@2ndquadrant.com
> On 02/08/2016 03:01 PM, Shulgin, Oleksandr wrote:
>> I've incorporated this fix into the v2 of my patch, I think it is
>> related closely enough. Also, added corresponding changes to
>> compute_distinct_stats(), which doesn't produce a histogram.
> I think it'd be useful not to have all the changes in one lump, but
> structure this as a patch series with related changes in separate chunks. I
> doubt we'd like to mix the changes in a single commit, and it makes the
> reviews and reasoning easier. So those NULL-handling fixes should be in one
> patch, the MCV patches in another one.
OK, such a split would make sense to me. Though, I'm a bit late as the
commitfest is already closed to new patches, I guess asking the CF manager
to split this might work (assuming I produce the patch files)?