Amit Langote wrote: > Anyway, after reading your replies, I thought of taking a stab at unifying > the partitioning information that's cached by relcache.c.
Wow. Now that's one large patch. I'm going to run with this for HEAD, but I think we should do a minimal fix for PG10. Did you detect any further bugs, while doing all this rework, apart from the one that started this thread? If not, I would prefer to do commit the minimal fix at start of thread to both branches, then apply the larger restructuring patch to HEAD only. For the record, I don't like the amount of code that this is putting in relcache.c. I am thinking that most of that code will go to src/backend/partitioning/partbounds.c instead. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
