<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

  <title></title>

</head>

<body bgcolor="#ffffff" text="#000000">

Max Kanat-Alexander wrote:

<blockquote cite="mid20070111014018.C2AEE18170@help.trusthosting.net"

 type="cite">

  <pre wrap="">On Wed, 10 Jan 2007 14:39:29 -0700 "Kevin Benton"

<a class="moz-txt-link-rfc2396E" href="mailto:kevin.benton@amd.com"><kevin.benton@amd.com></a> wrote:

  </pre>

  <blockquote type="cite">

    <pre wrap="">I'm wondering, why do we define the elements of the component table 

twice?  Why not just use the definition from Bugzilla/DB/Schema.pm ?

    </pre>

  </blockquote>

  <pre wrap=""><!---->

    Look at DB_COLUMNS in every single Bugzilla::Object derivative, and

see if they all match up with the exact schema. Bugzilla::Bug is a

great example.

        But it might be good to have a default for DB_COLUMNS that

reads from bz_schema if it's not specified.

        -Max

  </pre>

</blockquote>

<font face="Helvetica, Arial, sans-serif">If nothing else, when a

derivative is not required, only specifying schema items once

(presumably in Schema.pm) certainly makes updating the schema and

interactions easier. :)  I suggest we work toward removing duplicate

definitions like the example I gave originally for code maintainability

purposes. :)  If nothing else, it seems that we ought to at least ask

Schema.pm for the type definition as opposed to re-coding it again in

another area - again for maintainability reasons.<br>

<br>

I also wonder if moving the database upgrade section of checksetup.pl

into Schema.pm also makes sense so it's all in one place, possibly to

Schema::check_and_upgrade_db().  Again, I say this because I'm thinking

about maintainability - if all the core Schema changes are limited to

Schema.pm, I believe that it should make it easier for maintainers to

keep the number of files they touch down when making changes to the

schema itself.<br>

<br>

Back to my version per component work... :)<br>

<br>

Kevin<br>

</font><br>

<div class="moz-signature">-- <br>

<img src="cid:part1.07060204.00050202@amd.com" border="0"></div>

</body>

</html>