On Mon, Jun 26, 2000 at 05:36:00PM -0700, Jon Stevens wrote:
> on 6/26/2000 4:04 PM, "Nissim" <[EMAIL PROTECTED]> wrote:
> 
> >> /**
> >> * Returns false because transactions are only necessary in postgres.
> >> */
> >> public boolean objectDataNeedsTrans()
> >> {
> >> return false;
> >> }
> 
> instead of doing this, why not just test to see if the Connection object is
> from postgres? seems easier.

-1

The DB adaptor classes are there precisely to avoid having mucky
DB-specific code in BasePeer. 

Even if it seems appropriate for this example to hardcode knowledge into
BasePeer, it sets a bad precedent.

-- 
Sean Legassick
[EMAIL PROTECTED]


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to