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

Re: Vacuum full warning

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: Vacuum full warning
Date: 2011-01-21 14:09:07
Message-ID: AANLkTin=R1NP_WPNUC_JEsCYgRLtLNxZJBC+7DP-vuyg@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On Fri, Jan 21, 2011 at 15:01, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> On Fri, Jan 21, 2011 at 1:46 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> On Wed, Jan 19, 2011 at 13:30, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>> On Wed, Jan 19, 2011 at 12:29 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>> On Wed, Jan 19, 2011 at 13:22, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>> On Wed, Jan 19, 2011 at 12:20 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>>>> I came across yet another case of "i did vacuum full because it seemed
>>>>>> like a good idea" today.. Urgh.
>>>>>>
>>>>>> What do you guys think of a patch like the attached, which throws a
>>>>>> warning when you run vacuum full from the maintenance dialog on a
>>>>>> pre-9.0 version?
>>>>>
>>>>> Something like that should be a guru hint shouldn't it?
>>>>
>>>> Hmm. That might be an idea. I always turn those off because the
>>>> majority of them are just annoying ;)
>>>>
>>>> Where the heck is the *source* for the guru hints stored?
>>>
>>> docs/xxxx/hints.
>>
>> Meh. So why didn't mym "git grep" find that.. *annoyance*.
>>
>> Anyway. Something like this, then? Am I missing something still?
>
> At least a change to the installer source to add the new file. Do we
> need something for Makefile.am as well, or are we using a wildcard
> there?

Ah, wasn't aware we used the installer from there. But yeah, other
ones are in the wxs file - will go add there.

Makefile.am appears to be using a wildcard, yes.

Q: what's our taken on batckpatching something like this to 1.12?

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

In response to

Responses

pgadmin-hackers by date

Next:From: Dave PageDate: 2011-01-21 14:11:21
Subject: Re: Vacuum full warning
Previous:From: Dave PageDate: 2011-01-21 14:01:44
Subject: Re: Vacuum full warning

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