|
|
Re: [CDO] Increasing prefetch performance [message #1848274 is a reply to message #1848214] |
Fri, 26 November 2021 10:22 |
Robert Schulk Messages: 146 Registered: July 2015 |
Senior Member |
|
|
Using the IQueryHandler seems very promising!
I am not finding any information on the CDORevisionInterner though. Is it spelled differently, maybe? Also: if we are returning the actual revisions, then it may become a bit complicated regarding resource based security (which we are using).
I tried the following approach: only return the CDOIDs of the objects that are required and then load them in the client via: CDOView.getObjects. But: for me, this method only returns a small fraction of the objects that I am trying to load. I am providing a list of ~800 objects and only get 4 loaded revisions as the return map. Could there be a bug in CDOView.getObjects method? (I am using R20210908-0655)
Edit: my bad! The CDOView.getObjects method does not return the newly loaded objects, but it caches them nevertheless! The performance is really good now!
Edit2: can I also prefetch all lock states? Calling CDOObject.cdoWriteLock().isLocked() still needs a client<->server roundtrip.
[Updated on: Fri, 26 November 2021 13:00] Report message to a moderator
|
|
|
Re: [CDO] Increasing prefetch performance [message #1848375 is a reply to message #1848274] |
Tue, 30 November 2021 09:39 |
|
The CDORevisionInterner was only recently added as a replacement of CDORevisionAdder, see commit 694c3178f66a66ae6813e0c57b8214fb033e1177. You'd call:revision = revisionManager.getCache().internRevision(revision);
Proper permissions are automatically applied by the server while sending them to the client, see BaseCDORevision.write(CDODataOutput, int, CDOBranchPoint) .
Regarding lock state prefetching as part of revision loading, I've just implemented that: https://bugs.eclipse.org/bugs/show_bug.cgi?id=577319 . See also See InternalCDORevisionManager.prefetchRevisions(CDOID, CDOBranchPoint, int, boolean, Consumer<CDORevision>) .
Cheers
/Eike
----
http://www.esc-net.de
http://thegordian.blogspot.com
http://twitter.com/eikestepper
|
|
|
|
Powered by
FUDForum. Page generated in 0.03773 seconds