Mailing-List: contact empire-db-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: empire-db-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of benjamin.venditti@arcor.de
 designates 151.189.21.45 as permitted sender)
Message-ID: <4E0FD375.1030303@arcor.de>
Date: Sun, 03 Jul 2011 04:27:01 +0200
From: Benjamin Venditti <benjamin.venditti@arcor.de>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; de;
 rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11
MIME-Version: 1.0
To: Matt DeHoust <mdehoust@gmail.com>,
 empire-db-dev@incubator.apache.org, empire-db-user@incubator.apache.org
Subject: Re: Relationships
References: <BANLkTim=z=tkO6g=efWO21cURtnBBav+3Q@mail.gmail.com>
	<4E0A67DA.5070509@arcor.de>
	<B297D4737108A0459B66C3A85BB7F90133AE46@kn.esteam.de>
	<BANLkTin=joxHTjY-ZKZ8X-ygwUa60VBHXg@mail.gmail.com>
	<4E0F4910.3070707@arcor.de>
 <BANLkTikh4WRNoQpW_6W6QsqPe51wc84L6w@mail.gmail.com>
In-Reply-To: <BANLkTikh4WRNoQpW_6W6QsqPe51wc84L6w@mail.gmail.com>
Content-Type: multipart/alternative;
 boundary="------------080808050408000301050701"

--------------080808050408000301050701
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Hi Matt,
your testcases will surely be valuable, as i havn't tested my code 
programatically!
I'll have a look at the test cases when you supply them.

Cherio
     Benjamin

Am 02.07.2011 23:11, schrieb Matt DeHoust:
> Benjamin,
>
> I had started implementing this feature myself before I saw your 
> message. The good news is the test case I wrote passes with your 
> implementation. If you'd like to add the test to the code base, I'd be 
> happy to attach a patch to the issue.
>
> I'm going to give it a try against MySQL using the maven plugin. I'll 
> let you know how it goes.
>
> Thanks,
> Matt
>
> On Sat, Jul 2, 2011 at 12:36 PM, Benjamin Venditti 
> <benjamin.venditti@arcor.de <mailto:benjamin.venditti@arcor.de>> wrote:
>
>     Hi there,
>
>     i just committed a working solution for EMPIREDB-110 the
>     reverse-engineering of foreign-key relationships.
>     i'd be happy if anyone could have a look at it.
>
>
>     Cherio
>        Benjamin
>
>     Am 29.06.2011 15:12, schrieb Matt DeHoust:
>
>         Thank you Benjamin and Rainer. I have subscribed to the dev
>         list. I'll take
>
>         a look at the code when I return from vacation and follow up
>         using the dev
>         list with any questions or ideas. I hope to have a chance to
>         look into it
>         this weekend.
>
>         -Matt
>
>         On Wed, Jun 29, 2011 at 6:30 AM, Rainer
>         D�bele<doebele@esteam.de <mailto:doebele@esteam.de>>  wrote:
>
>             Hi everyone,****
>
>             ** **
>
>             I think this is an important improvement of the code
>             generator that we
>             should implement.****
>
>             I have created a Jira Issue for this:****
>
>             ** **
>
>             https://issues.apache.org/jira/browse/EMPIREDB-110****
>
>             ** **
>
>             Matt, if you feel you can and want to do It, then we would
>             certainly
>             appreciate this.****
>
>             You may create a patch file and attach it to the JIRA
>             ticket.****
>
>             Then we can apply the patch and commit the code to svn.****
>
>             ** **
>
>             For further discussion I would recommend to move to the
>             dev-mailing list:*
>             ***
>
>             empire-db-dev@incubator.apache.org
>             <mailto:empire-db-dev@incubator.apache.org>****
>
>             ** **
>
>             Thanks, ****
>
>             Rainer****
>
>             ** **
>
>             ** **
>
>             *Von:* Benjamin Venditti
>             [mailto:benjamin.venditti@arcor.de
>             <mailto:benjamin.venditti@arcor.de>]
>             *Gesendet:* Mittwoch, 29. Juni 2011 01:47
>             *An:* empire-db-user@incubator.apache.org
>             <mailto:empire-db-user@incubator.apache.org>
>             *Cc:* Matt DeHoust
>             *Betreff:* Re: Relationships****
>
>             ** **
>
>             Hi Matt,
>
>             thanks for your interest in empire-db.
>
>             regarding your request ... unfortunately the codegen does
>             not set up the
>             relation of the database model.
>             At the moment the codegen will only set up views and
>             tables. We thought
>             these are the most important for the most people as they
>             are usually
>             sufficient for simple data access.
>
>             However i think there are two occasions where the
>             relations are important:
>                 - database creation
>                 - deletion of records with cascading deletion enabled
>
>             At the moment we have not planned to add "relationship
>             generation"  to the
>             codegen component, but I am glad to hear that you consider
>             to extend it on
>             your own.
>
>             The right place to start with that would be:
>                * CodeGenParser.populateDatabase(..) : 131* : This is
>             the point where
>             the DatabaseMetaData is querried and used to populate
>             empire-db's model
>                 *CodeGenWriter.generateCodeFiles(..) : 125* : Is used
>             to serialize the
>             populated model as java code
>                 *src/main/resources/Database.vm :  70* : This is the
>             velocity template
>             we use to create the java class for the database.
>
>             In its default constructor you'd need to add a line for
>             each relationship
>             similar to this:
>
>                                             *addRelation(
>             EMPLOYEES.DEPARTMENT_ID.referenceOn(
>             DEPARTMENTS.DEPARTMENT_ID ));*
>
>             I hope this is of some help to you. Feel free to ask if
>             you have further
>             questions!
>
>             Cherio
>                 Benjamin
>
>             Am 28.06.2011 19:37, schrieb Matt DeHoust: ****
>
>             Hello, ****
>
>             ** **
>
>             I see that the empire-db model supports relationships (for
>             example
>             DBDatabase.getRelations()), but it doesn't seem that the
>             codegen component
>             sets them up. Am I missing something? (I am using MySQL if
>             that matters.)*
>             ***
>
>             ** **
>
>             If not, are there plans to introduce relationship setup in
>             the codegen
>             component?****
>
>             ** **
>
>             If not, where should I look to add them myself?
>             CodeGenParser?****
>
>             ** **
>
>             Thanks,****
>
>             Matt****
>
>             ** **
>
>             P.S. I apologize if I posted this question twice by
>             mitake.****
>
>             ** **
>
>
>


--------------080808050408000301050701--