Re: that picksplit debug message again - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: that picksplit debug message again
Date
Msg-id 1245922566.4038.62.camel@ebony.2ndQuadrant
Whole thread Raw
In response to Re: that picksplit debug message again  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Wed, 2009-06-24 at 01:19 -0400, Tom Lane wrote:
> Peter Eisentraut <peter_e@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.comPostgreSQL Training, Services and Support



pgsql-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: Extensions User Design
Next
From: Magnus Hagander
Date:
Subject: Re: Extensions User Design