Skip site navigation (1) Skip section navigation (2)

Re: Two strange settings in frmOptions

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Two strange settings in frmOptions
Date: 2011-12-12 14:17:52
Message-ID: CABUevEyG3ysG8xa0aOWZJzA8Uv8iac_g9__QDfAmQYWg=+HkRg@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On Mon, Dec 12, 2011 at 15:13, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> On Mon, Dec 12, 2011 at 15:08, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> On Mon, Dec 12, 2011 at 7:27 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>> Noticed two things in Options->QueryTool:
>>>
>>> 1) There is a settin gfor "Maximum rows to retreive". From what I can
>>> tell, this is *only* used for pgagent (pgaJobs.cpp and pgaStep.cpp).
>>> If that is so, it shuld not be under Query Tool settings. Potentially
>>> it could be removed completely, if pgAgent can actually deal with a
>>> hardcode there?
>>
>> It's not that pgAgent can or can't cope - it's just there to put a
>> limit on the number of rows we try to load into the Statistics tab on
>> the main form. Agreed, it shouldn't be under Query Tool.
>
> I meant if we can get away with a hardcoded limit - e.g. make it
> always do LIMIT 100.
>
>
>>> 2) There is a setting for "Count rows if estimated less than". AFAICT,
>>> this again has nothing to do with the query tool - this is for the
>>> object browser, when you click a table. Shouldn't it be moved to be
>>> under another tab?
>>
>> +1
>
> Any suggestions for where?

Argh. I suck. Guillaume has already fixed this in HEAD. I thought I
was running HEAD, but had some total weirdness in my PATH...

-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

In response to

pgadmin-hackers by date

Next:From: Dave PageDate: 2011-12-12 14:18:48
Subject: Re: Two strange settings in frmOptions
Previous:From: Dave PageDate: 2011-12-12 14:15:09
Subject: Re: pgagent extension

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group