On Tue, Aug 13, 2019 at 03:08:34PM +0530, amul sul wrote: > On Sat, Aug 10, 2019 at 12:16 AM David Fetter <da...@fetter.org> wrote: > > > On Thu, Aug 08, 2019 at 05:42:21PM +0900, Amit Langote wrote: > > > On Thu, Aug 8, 2019 at 5:33 PM amul sul <sula...@gmail.com> wrote: > > > > On Thu, Aug 8, 2019 at 1:27 PM Amit Langote <amitlangot...@gmail.com> > > wrote: > > > > > > >> Thanks for the patch. This was discussed recently in the "hyrax vs. > > > >> RelationBuildPartitionDesc()" thread [1] and I think Alvaro proposed > > > >> an approach that's similar to yours. Not sure why it wasn't pursued > > > >> though. Maybe the reason is buried somewhere in that discussion. > > > > > > > > Oh, quite similar, thanks Amit for pointing that out. > > > > > > > > Look like "hyrax vs.RelationBuildPartitionDesc()" is in discussion for > > the master > > > > branch only, not sure though, but we need the similar fix for the back > > branches as well. > > > > > > Well, this is not a bug as such, so it's very unlikely that a fix like > > > this will be back-patched. Also, if this becomes an issue only for > > > more than over 1000 partitions, then it's not very relevant for PG 10 > > > and PG 11, because we don't recommend using so many partitions with > > > them. Maybe we can consider fixing PG 12 though. > > > > A fix for the thousands-of-partitions case would be very welcome for 12. > > > > > Look like commit # d3f48dfae42 added the required fix but is enabled only > for > the clobber-cache builds :(
I've got a real world multi-tenancy case that would really be helped by this. Can we enable it for all builds, please? Best, David. -- David Fetter <david(at)fetter(dot)org> http://fetter.org/ Phone: +1 415 235 3778 Remember to vote! Consider donating to Postgres: http://www.postgresql.org/about/donate