On Tue, Mar 21, 2017 at 12:49 PM, Bruce Momjian <br...@momjian.us> wrote: > On Tue, Mar 21, 2017 at 09:25:49AM -0400, Robert Haas wrote: >> On Tue, Mar 21, 2017 at 8:41 AM, Pavan Deolasee >> > TBH I see many artificial scenarios here. It will be very useful if he can >> > rerun the query with some of these restrictions lifted. I'm all for >> > addressing whatever we can, but I am not sure if this test demonstrates a >> > real world usage. >> >> That's a very fair point, but if these patches - or some of them - are >> going to get committed then these things need to get discussed. Let's >> not just have nothing-nothing-nothing giant unagreed code drop. > > First, let me say I love this feature for PG 10, along with > multi-variate statistics. > > However, not to be a bummer on this, but the persistent question I have > is whether we are locking ourselves into a feature that can only do > _one_ index-change per WARM chain before a lazy vacuum is required. Are > we ever going to figure out how to do more changes per WARM chain in the > future, and is our use of so many bits for this feature going to > restrict our ability to do that in the future. > > I know we have talked about it, but not recently, and if everyone else > is fine with it, I am too, but I have to ask these questions.
I think that's a good question. I previously expressed similar concerns. On the one hand, it's hard to ignore the fact that, in the cases where this wins, it already buys us a lot of performance improvement. On the other hand, as you say (and as I said), it eats up a lot of bits, and that limits what we can do in the future. On the one hand, there is a saying that a bird in the hand is worth two in the bush. On the other hand, there is also a saying that one should not paint oneself into the corner. I'm not sure we've had any really substantive discussion of these issues. Pavan's response to my previous comments was basically "well, I think it's worth it", which is entirely reasonable, because he presumably wouldn't have written the patch that way if he thought it sucked. But it might not be the only opinion. -- Robert Haas EnterpriseDB: 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
