On Thu, Jul 7, 2011 at 2:43 PM, Robert Haas <[email protected]> wrote: > On Wed, Jul 6, 2011 at 2:50 PM, Noah Misch <[email protected]> wrote: >> Drat; fixed in this version. My local branches contain a large test battery >> that I filter out of the diff before posting. This time, that filter also >> removed an essential part of the patch. > > OK, I'm pretty happy with this version, with the following minor caveats: > > 1. I still think the documentation changes could use some > word-smithing. If I end up being the one who commits this, I'll take > a look at that as part of the commit. > > 2. I think it would be helpful to add a comment explaining why > CheckIndexCompatible() is calling
Woops, sorry. Hit send too soon. ...why CheckIndexCompatible() is calling ComputeIndexAttrs(). Rather than committing this immediately, I'm going to mark this "Ready for Committer", just in case Tom or someone else wants to look this over and weigh in. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
