I don't think you overlooked it.  I think it never got posted (could be our
outgoing mailer doing stupid things).

Well, the performance issue would depend on how many obects you're
deleting.  If you're deleting hundreds of objects in a transaction, there
might be a difference, though it might not be a big one.

Glad to see a fix coming.

(Open source is GREAT, ain't it?)

David



|---------+--------------------------->
|         |           "Armin Waibel"  |
|         |           <[EMAIL PROTECTED]|
|         |           ait.de>         |
|         |                           |
|         |           07/08/2003 08:57|
|         |           AM              |
|         |           Please respond  |
|         |           to "OJB Users   |
|         |           List"           |
|         |                           |
|---------+--------------------------->
  
>--------------------------------------------------------------------------------------------------------------------------------------------------|
  |                                                                                    
                                                              |
  |        To:      "OJB Users List" <[EMAIL PROTECTED]>                               
                                                         |
  |        cc:                                                                         
                                                              |
  |        Subject: Re: PersistenceBrokerImpl.delete()                                 
                                                              |
  |                                                                                    
                                                              |
  
>--------------------------------------------------------------------------------------------------------------------------------------------------|



Hi David,

sorry for overlook your post twice,
such was not my intention.
I think you are right - will fix this ASAP!

> 1) Shouldn't this be a set, for perfomance reasons.
hmm, in that case I don't think List is an performance impact.

> 2) Should this be based on Identity, rather than on the object (and
it's
> equals method) itself?
I fully agree with that.

regards,
Armin


----- Original Message -----
From: <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Tuesday, July 08, 2003 2:36 PM
Subject: PersistenceBrokerImpl.delete()


> I posted this (twice) to the DEV list, but never saw it come
through....
>
> PB keeps a list of objects that are marked for delete, and avoids
deleting
> them twice.
>
> 1) Shouldn't this be a set, for perfomance reasons.
> 2) Should this be based on Identity, rather than on the object (and
it's
> equals method) itself?
>
> David
>
>
> This message contains information from Equifax Inc. which may be
> confidential and privileged.  If you are not an intended recipient,
please
> refrain from any disclosure, copying, distribution or use of this
> information and note that such actions are prohibited.  If you have
> received this transmission in error, please notify by e-mail
> [EMAIL PROTECTED]
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]







---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to