>> I also noticed that this patch makes matview test fail. It seems >> that it just changes the ordering of rows for queries like >> "SELECT * FROM tv;". Does this seem like a bug in my patch, or >> should we add "ORDER BY" clauses to this test to make it more >> deterministic?
> I added some ORDER BY clauses. That is probably a good thing > anyway for purposes of code coverage. Does that fix it for you?
Uh, what? Fooling around with the implementation of avg() should surely not change any planning decisions.