Re: pg_advisor schema proof of concept - Mailing list pgsql-hackers

From Robert Treat
Subject Re: pg_advisor schema proof of concept
Date
Msg-id 1080251973.11890.485.camel@camel
Whole thread Raw
In response to Re: pg_advisor schema proof of concept  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: pg_advisor schema proof of concept
Re: pg_advisor schema proof of concept
List pgsql-hackers
On Thu, 2004-03-25 at 11:31, Fabien COELHO wrote:
> 
> > > Also, if they have a partial index on the FK, it's not good enough!  In
> > > CVS, IS NOT NULL partial indexes should be used, but in general all
> > > others still won't...
> >
> > Whoa, there, partner!   Keep in mind that there are *often* reasons for using
> > a partial index on an FK, or even no index at all!  The docs for pg_advisor
> > need to reflect that it only catches little details the developer might
> > otherwise have missed.   It's not smarter than a DBA.
> 
> Sure.
> 
> That's why advices are "graded" from info to error in the current
> preliminary version.
> 
> Advices that may or may not be good depending on undecidable elements
> have a lower grade. For instance, most attributes should be "NOT NULL"
> from a statistical point of view, but it is perfectly legitimate to
> have nullable attributes mostly anywhere, so the corresponding advices
> is just an "info".
> 

Are you planning on making some type of differentiation on advise that
is performance based rather than advise that is theory based?  I see
both cases being hinted at and it seems like a subtle but important
piece of information...

Robert Treat
-- 
Build A Brighter Lamp :: Linux Apache {middleware} PostgreSQL



pgsql-hackers by date:

Previous
From: David Fetter
Date:
Subject: Re: HEAD compile troubles
Next
From: Bruce Momjian
Date:
Subject: Re: Log rotation