On Wed, 2009-06-24 at 01:19 -0400, Tom Lane wrote: > Peter Eisentraut <pete...@gmx.net> writes: > > Is the %d actually in the right place here? > > errmsg("picksplit method for %d column of index \"%s\" failed", > > attno + 1, RelationGetRelationName(r)) > > No, any native speaker of English would say "for column %d". Putting > "failed" at the end seems a bit awkward as well, though I can't offhand > see a better phrasing. "picksplit method failed for ..." is *not* > better; it implies there is only one picksplit method for everything, > whereas the point of the message is that the one associated with this > column failed.
"has failed"? -- Simon Riggs www.2ndQuadrant.com PostgreSQL Training, Services and Support -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers