[Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-25 Thread Tobias Schlitt
Hi! I just went through the design draft I wrote together with Bertrand for 2 new features we plan to implement in PersistentObject 1.5 (2008.2): - Identity map - Relation pre-fetching The identity map feature will avoid to have 2 different objects representing the same database data in one requ

[Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-25 Thread Tobias Schlitt
Hi! I just went through the design draft I wrote together with Bertrand for 2 new features we plan to implement in PersistentObject 1.5 (2008.2): - Identity map - Relation pre-fetching The identity map feature will avoid to have 2 different objects representing the same database data in one requ

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-27 Thread Derick Rethans
On Wed, 25 Jun 2008, Tobias Schlitt wrote: > Please review the design draft and post your comments here. Some comments: Inside this method, first it should be checked if the same find query was already performed earlier. In this case, a cached result set can be returned. You can't reall

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-27 Thread James Pic
Hi Tobias! > The Identity Map support should be optional to not break BC and keep > flexibility. Therefore, a new class named ezcPersistentIdentityMapSession is > implemented. This extends the current implementation ezcPersistentSession to > make instanceof checks still work. All of the methods wi

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-27 Thread Jared Williams
> -Original Message- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of Derick Rethans > Sent: 27 June 2008 09:38 > To: Tobias Schlitt > Cc: Bertrand Dunogier; Components > Subject: Re: [Components] [PersistentObject] Design: Identity > map a

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-28 Thread Tobias Schlitt
Hi! On 06/27/2008 10:38 AM Derick Rethans wrote: > On Wed, 25 Jun 2008, Tobias Schlitt wrote: >> Please review the design draft and post your comments here. > Some comments: > Inside this method, first it should be checked if the same find query > was already performed earlier. In this cas

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-28 Thread Tobias Schlitt
Hi James! On 06/27/2008 11:44 AM James Pic wrote: >> The Identity Map support should be optional to not break BC and keep >> flexibility. Therefore, a new class named ezcPersistentIdentityMapSession is >> implemented. This extends the current implementation ezcPersistentSession to >> make instanc

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-28 Thread Tobias Schlitt
Hi Jared! On 06/27/2008 12:33 PM Jared Williams wrote: >> On Wed, 25 Jun 2008, Tobias Schlitt wrote: >>> Please review the design draft and post your comments here. >> Some comments: >> >> Inside this method, first it should be checked if the same >> find query >> was already performed ear

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-28 Thread Jared Williams
> -Original Message- > From: Tobias Schlitt [mailto:[EMAIL PROTECTED] > Sent: 28 June 2008 17:38 > To: Jared Williams > Cc: 'Derick Rethans'; 'Bertrand Dunogier'; 'Components' > Subject: Re: [Components] [PersistentObject] Design: I

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-29 Thread Gaetano Giunta
Jared Williams a écrit : -Original Message- From: Tobias Schlitt [mailto:[EMAIL PROTECTED] Sent: 28 June 2008 17:38 To: Jared Williams Cc: 'Derick Rethans'; 'Bertrand Dunogier'; 'Components' Subject: Re: [Components] [PersistentObject] Design:

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-29 Thread Jared Williams
> Oracle does not have a "rowid" id/timestamp for this (oracle > standard rowid pseudocolumn is actually a row identifier, not > row version identifier, guaranteed to be unique only within > the table itself). > > In Oracle 11g there is the ORA_ROWSCN that could be used to > identify this

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-06-30 Thread Tobias Schlitt
Hi! I updated the identity map design after discussion here on the list as follows: - Removed the query caching (due to problems with bound paramaters). - Changed the ezcPersistentIdentitySession to be a decorator for ezcPersistentSession instead of extending it. - Removed the $refetch paramete

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-01 Thread James Pic
Hi Tobias! Tobias Schlitt wrote: > On 06/27/2008 11:44 AM James Pic wrote: > >> The Identity Map support should be optional to not break BC and keep > >> flexibility. Therefore, a new class named ezcPersistentIdentityMapSession > >> is > >> implemented. This extends the current implementation ezc

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-02 Thread Tobias Schlitt
Hi James! On 07/01/2008 05:53 PM James Pic wrote: > Tobias Schlitt wrote: >> On 06/27/2008 11:44 AM James Pic wrote: The Identity Map support should be optional to not break BC and keep flexibility. Therefore, a new class named ezcPersistentIdentityMapSession is implemented.

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-02 Thread James PIC
Hi Tobias! On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: > > On 07/01/2008 05:53 PM James Pic wrote: >> Tobias Schlitt wrote: >>> On 06/27/2008 11:44 AM James Pic wrote: > > The Identity Map support should be optional to not break BC and keep > flexibility. Ther

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-03 Thread Tobias Schlitt
Hi James! On 07/02/2008 02:11 PM James PIC wrote: > On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: >> On 07/01/2008 05:53 PM James Pic wrote: >>> Tobias Schlitt wrote: On 06/27/2008 11:44 AM James Pic wrote: >> The Identity Map support should be optional to not

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-04 Thread James Pic
Hi Tobias! Tobias Schlitt wrote: > On 07/02/2008 02:11 PM James PIC wrote: > > On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: > >> On 07/01/2008 05:53 PM James Pic wrote: > >>> Tobias Schlitt wrote: > On 06/27/2008 11:44 AM James Pic wrote: > > >> The Identity M

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-04 Thread Tobias Schlitt
On 07/04/2008 01:25 PM James Pic wrote: > Hi Tobias! > Tobias Schlitt wrote: >> On 07/02/2008 02:11 PM James PIC wrote: >>> On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: On 07/01/2008 05:53 PM James Pic wrote: This should be easy to implement by replacing th

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-07-06 Thread James Pic
Hi Tobias! Tobias Schlitt wrote: > On 07/04/2008 01:25 PM James Pic wrote: > > Tobias Schlitt wrote: > >> On 07/02/2008 02:11 PM James PIC wrote: > >>> On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: > On 07/01/2008 05:53 PM James Pic wrote: > >> If it was for me, we

Re: [Components] [PersistentObject] Design: Identity map and relation pre-fetching support

2008-08-02 Thread Tobias Schlitt
Hi! On 07/06/2008 11:29 AM James Pic wrote: > Tobias Schlitt wrote: >> On 07/04/2008 01:25 PM James Pic wrote: >>> Tobias Schlitt wrote: On 07/02/2008 02:11 PM James PIC wrote: > On Wed, Jul 2, 2008 at 9:19 AM, Tobias Schlitt <[EMAIL PROTECTED]> wrote: >> On 07/01/2008 05:53 PM James