Bug 870330

Summary:	Cache records in memory
Product:	[Retired] oVirt	Reporter:	Laszlo Hornyak <lhornyak>
Component:	ovirt-engine-core	Assignee:	Liran Zelkha <lzelkha>
Status:	CLOSED DUPLICATE	QA Contact:	bugs <bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	unspecified	CC:	bazulay, bsettle, bugs, dfediuck, emesika, oourfali, rbalakri
Target Milestone:	---
Target Release:	3.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	infra
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-08-11 08:41:31 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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 ***