Eclipse Community Forums: EMF "Technology" (Ecore Tools, EMFatic, etc)

Help

Home

Home » Modeling » EMF "Technology" (Ecore Tools, EMFatic, etc) » [CDO][0.8.0] Concurrency

Show: Today's Messages :: Show Polls :: Message Navigator

[CDO][0.8.0] Concurrency [message #106101]

Fri, 04 January 2008 15:34

Simon Mc Duff

Messages: 596
Registered: July 2009

Senior Member

I updating our prodcution system with the latest version of CDO.
(we are still using the version from october)

I'm going through all the changes we've made... and it is funny that we
found the same fixes at many places.

One of them.. we have almost the same change.

The problem with CDOViewImpl.getObject() its that we need to put
synchronization around lastLookup and when we assign the lastLookup

if (id.equals(lastLookupID))
{
<< CONTEXT COULD SWITCH HERE !!!
return lastLookupObject;
}

Let me know what you think.

Attachment: CDOViewImpl-GetObject.patch
(Size: 2.10KB, Downloaded 215 times)

Report message to a moderator

Re: [CDO][0.8.0] Concurrency [message #106170 is a reply to message #106101]

Fri, 04 January 2008 17:27

Eclipse User

Originally posted by: stepper.sympedia.de

Simon,

That's a good idea. I was too eager with sparing synced statements ;-(
https://bugs.eclipse.org/bugs/show_bug.cgi?id=214370

Regards,
Eike Stepper
----
http://wiki.eclipse.org/CDO
http://wiki.eclipse.org/Net4j

Simon McDuff schrieb:
> I updating our prodcution system with the latest version of CDO.
> (we are still using the version from october)
>
> I'm going through all the changes we've made... and it is funny that we
> found the same fixes at many places.
>
> One of them.. we have almost the same change.
>
> The problem with CDOViewImpl.getObject() its that we need to put
> synchronization around lastLookup and when we assign the lastLookup
>
> if (id.equals(lastLookupID))
> {
> << CONTEXT COULD SWITCH HERE !!!
> return lastLookupObject;
> }
>
> Let me know what you think.
>
>
>

Report message to a moderator

Re: [CDO][0.8.0] Concurrency [message #613918 is a reply to message #106101]

Fri, 04 January 2008 17:27

Eike Stepper

Messages: 6682
Registered: July 2009

Senior Member

Simon,

That's a good idea. I was too eager with sparing synced statements ;-(
https://bugs.eclipse.org/bugs/show_bug.cgi?id=214370

Regards,
Eike Stepper
----
http://wiki.eclipse.org/CDO
http://wiki.eclipse.org/Net4j

Simon McDuff schrieb:
> I updating our prodcution system with the latest version of CDO.
> (we are still using the version from october)
>
> I'm going through all the changes we've made... and it is funny that we
> found the same fixes at many places.
>
> One of them.. we have almost the same change.
>
> The problem with CDOViewImpl.getObject() its that we need to put
> synchronization around lastLookup and when we assign the lastLookup
>
> if (id.equals(lastLookupID))
> {
> << CONTEXT COULD SWITCH HERE !!!
> return lastLookupObject;
> }
>
> Let me know what you think.
>
>
>

Cheers
/Eike

----
http://www.esc-net.de
http://thegordian.blogspot.com
http://twitter.com/eikestepper

Report message to a moderator

Previous Topic:	[Compare] using separate model resource sets for left/right/ancestor
Next Topic:	[CDO][0.8.0] Problem with OutOfMemory

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Fri Apr 19 10:08:04 GMT 2024

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter