Yair Zaslavsky has posted comments on this change.

Change subject: engine : Introduction of CommandManager
......................................................................


Patch Set 2: (1 inline comment)

Ravi, already an important comment - can we have some info on why you chose 
EHCache vs other solutions - i.e infinispan?

....................................................
File backend/manager/modules/bll/pom.xml
Line 118:       <scope>test</scope>
Line 119:     </dependency>
Line 120: 
Line 121:     <dependency>
Line 122:       <groupId>net.sf.ehcache</groupId>
Tabs, please remove them.
Line 123:       <artifactId>ehcache</artifactId>
Line 124:       <version>2.6.5</version>
Line 125:         <type>pom</type>
Line 126:     </dependency>


--
To view, visit http://gerrit.ovirt.org/13152
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibdd7585bfcfa6adeb761a8532218ba1aaa5e3c5d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to