On Wed, Feb 24, 2016 at 12:30 AM, Tomas Vondra <tomas.von...@2ndquadrant.com > wrote:
> Hi, > > 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)? -- Alex
