On Wed, Mar 30, 2016 at 8:40 AM, Tomas Vondra <tomas.von...@2ndquadrant.com> wrote:
>> === >> @@ -2697,6 +2697,7 @@ check_partial_indexes(PlannerInfo *root, RelOptInfo >> *rel) >> continue; /* don't repeat >> work if already proven OK */ >> >> have_partial = true; >> + break; >> } >> if (!have_partial) >> return; >> === >> >> The initialization has been moved to set_rel_size so the break >> can be restored. > > FWIW the break was restored in the v9 by me. Yeah, I kept that in v10, so the three of us seem to be on the same page there. >> FWIW, as mentioned upthread, I added the following condition to >> decline ripping index predicates from base restrictinfo without >> understanding the reason to do so. > > Ummmm, I'm a bit confused. Which condition? Yeah, any additional discussion about areas which anyone sees as open or still needing attention might allow me to get enough traction to wrap this; I'm having trouble seeing what the pending issues are where both Tom and Kyotaro-san seemed to be unsatisfied. -- Kevin Grittner EDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
