Re: [HACKERS] enable_indexonly

2012-04-30 Thread Magnus Hagander
On Sat, Apr 28, 2012 at 17:41, Andrew Dunstan and...@dunslane.net wrote: On 04/27/2012 12:44 PM, Magnus Hagander wrote: Hmm. Forgive me, I pressed the wrong button and looked at current docs rather than dev docs. (Easier when they used to look different...) Maybe we should have the

Re: [HACKERS] enable_indexonly

2012-04-28 Thread Andrew Dunstan
On 04/27/2012 12:44 PM, Magnus Hagander wrote: Hmm. Forgive me, I pressed the wrong button and looked at current docs rather than dev docs. (Easier when they used to look different...) Maybe we should have the stylesheet watermark the dev docs pages. Are you offering to write the patch?

[HACKERS] enable_indexonly

2012-04-27 Thread Simon Riggs
I notice that there isn't a parameter called enable_indexonly (or similar). ISTM that such a major new feature should be controlled by a planner method parameter, just as all the existing planner methods are. This will help us evaluate index only scans in production, and turn them off if they

Re: [HACKERS] enable_indexonly

2012-04-27 Thread Thom Brown
On 27 April 2012 16:08, Simon Riggs si...@2ndquadrant.com wrote: I notice that there isn't a parameter called enable_indexonly (or similar). ISTM that such a major new feature should be controlled by a planner method parameter, just as all the existing planner methods are. This will help us

Re: [HACKERS] enable_indexonly

2012-04-27 Thread Simon Riggs
On Fri, Apr 27, 2012 at 4:41 PM, Thom Brown t...@linux.com wrote: On 27 April 2012 16:08, Simon Riggs si...@2ndquadrant.com wrote: I notice that there isn't a parameter called enable_indexonly (or similar). ISTM that such a major new feature should be controlled by a planner method parameter,

Re: [HACKERS] enable_indexonly

2012-04-27 Thread Andrew Dunstan
On 04/27/2012 11:45 AM, Simon Riggs wrote: On Fri, Apr 27, 2012 at 4:41 PM, Thom Brownt...@linux.com wrote: On 27 April 2012 16:08, Simon Riggssi...@2ndquadrant.com wrote: I notice that there isn't a parameter called enable_indexonly (or similar). ISTM that such a major new feature should

Re: [HACKERS] enable_indexonly

2012-04-27 Thread Simon Riggs
On Fri, Apr 27, 2012 at 4:56 PM, Andrew Dunstan and...@dunslane.net wrote: (Easier when they used to look different...) Maybe we should have the stylesheet watermark the dev docs pages. +1 Many users have 1 version in production. This would help --  Simon Riggs  

Re: [HACKERS] enable_indexonly

2012-04-27 Thread Magnus Hagander
On Fri, Apr 27, 2012 at 17:56, Andrew Dunstan and...@dunslane.net wrote: On 04/27/2012 11:45 AM, Simon Riggs wrote: On Fri, Apr 27, 2012 at 4:41 PM, Thom Brownt...@linux.com  wrote: On 27 April 2012 16:08, Simon Riggssi...@2ndquadrant.com  wrote: I notice that there isn't a parameter