Robert Haas wrote: > On Fri, Apr 6, 2018 at 8:24 AM, Alvaro Herrera <alvhe...@alvh.no-ip.org> > wrote: > > I don't actually like very much the idea of putting all this code in > > optimizer/util. This morning it occurred to me that we should create a new > > src/backend/partitioning/ (and a src/include/partitioning/ to go with > > it) and drop a bunch of files there. Even your proposed new partcache.c > > will seem misplaced *anywhere*, since it contains support code to be > > used by both planner and executor; in src/{backend,include}/partitioning > > it will be able to serve both without it being a modularity wart. > > Uh, what? > > Surely partcache.c is correctly placed next to relcache.c and > syscache.c and everything else in src/backend/utils/cache.
Frankly, I'm not real sure about partcache.c yet. Are you? -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services