Bug 870330 - Cache records in memory
Summary: Cache records in memory
Keywords:
Status: CLOSED DUPLICATE of bug 1159621
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.6.0
Assignee: Liran Zelkha
QA Contact: bugs@ovirt.org
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-10-26 07:54 UTC by Laszlo Hornyak
Modified: 2016-02-10 19:31 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-08-11 08:41:31 UTC
oVirt Team: Infra
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 21952 0 None None None Never

Description Laszlo Hornyak 2012-10-26 07:54:31 UTC
Description of problem:

Some database operations could be eliminated by keeping a cache for the records. This could shorten the engine response times and drop the database load.

For example VDS static records change very rarely, but they are used almost all the time by the engine, the number of such records rarely exceed some hundred, which is still a very small amount of space in the memory.

Comment 1 Itamar Heim 2012-10-28 08:46:44 UTC
we should not add caching until we solve jboss clustering aspects.
the idea of not having caches in engine (yes, we have a few culprits) is to allow distributing/partitioning the load of the engine to multiple processes.

Comment 2 Laszlo Hornyak 2012-10-29 07:06:52 UTC
I know there are some static Map's to cache data. Probably replacing those to any real cache implementations would help to solve the clustering problems. Probably any cache implementation can work on distributed mode.

Comment 3 Itamar Heim 2013-05-05 06:28:17 UTC
isn't the infra for this now done with the infinispan patches to close this?

Comment 4 Laszlo Hornyak 2013-05-05 09:39:59 UTC
(In reply to comment #3)
> isn't the infra for this now done with the infinispan patches to close this?

Last time I have seen the cache implementation it was not caching results from DAO's, but I will check again.

Comment 5 Laszlo Hornyak 2013-05-06 07:34:52 UTC
(In reply to comment #3)
> isn't the infra for this now done with the infinispan patches to close this?

Checked again on latest master and cache is not used for cahcing Db records.

Comment 7 Eli Mesika 2013-11-11 21:16:48 UTC
http://gerrit.ovirt.org/#/c/20894/ seems related to this BZ
Liran please handle

Comment 8 Liran Zelkha 2013-11-12 06:27:27 UTC
Please see gerrit change http://gerrit.ovirt.org/#/c/20894/

Comment 9 Sandro Bonazzola 2014-03-04 09:28:25 UTC
This is an automated message.
Re-targeting all non-blocker bugs still open on 3.4.0 to 3.4.1.

Comment 11 Oved Ourfali 2015-08-11 08:41:31 UTC

*** This bug has been marked as a duplicate of bug 1159621 ***


Note You need to log in before you can comment on or make changes to this bug.