From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
---|---|
To: | Josh Berkus <josh(at)agliodbs(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pg_advisor schema proof of concept |
Date: | 2004-03-25 16:31:15 |
Message-ID: | Pine.LNX.4.58.0403251726280.7217@sablons.cri.ensmp.fr |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> > 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".
--
Fabien Coelho - coelho(at)cri(dot)ensmp(dot)fr
From | Date | Subject | |
---|---|---|---|
Next Message | Devrim GUNDUZ | 2004-03-25 17:12:54 | Re: Email addresses on developer bios site |
Previous Message | Josh Berkus | 2004-03-25 16:19:09 | Re: pg_advisor schema proof of concept |