<div dir="ltr">Awesome!!!</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 1, 2013 at 10:29 AM, Allon Mureinik <span dir="ltr">&lt;<a href="mailto:amureini@redhat.com" target="_blank">amureini@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
----- Original Message -----<br>
&gt; From: &quot;Liran Zelkha&quot; &lt;<a href="mailto:liran.zelkha@gmail.com">liran.zelkha@gmail.com</a>&gt;<br>
&gt; To: &quot;Yair Zaslavsky&quot; &lt;<a href="mailto:yzaslavs@redhat.com">yzaslavs@redhat.com</a>&gt;<br>
&gt; Cc: &quot;Allon Mureinik&quot; &lt;<a href="mailto:amureini@redhat.com">amureini@redhat.com</a>&gt;, &quot;engine-devel&quot; &lt;<a href="mailto:engine-devel@ovirt.org">engine-devel@ovirt.org</a>&gt;<br>
&gt; Sent: Sunday, June 30, 2013 11:37:26 AM<br>
&gt; Subject: Re: [Engine-devel] Guid improvements<br>
&gt;<br>
&gt; Great news.<br>
&gt; Allon - please note that GUID is being recreated from String by both DB calls<br>
&gt; and by data received from VDSM. It is VERY useful to cache Guid<br>
&gt; String--&gt;Guid, and not just Empty GUID.<br>
</div>I generally agree about the high cost of sting&lt;-&gt;uuid operations, but I&#39;m not sure caching is the way to go, at least not everywhere.<br>
<br>
At least for the database, there is a much easier solution - stop using strings to represent uuids.<br>
Here&#39;s an example of how it can be done:<br>
<a href="http://gerrit.ovirt.org/#/c/16281/" target="_blank">http://gerrit.ovirt.org/#/c/16281/</a><br>
<div class="HOEnZb"><div class="h5"><br>
<br>
&gt;<br>
&gt; However, as the Guid class runs in GWT as well, you can&#39;t use Infinispan and<br>
&gt; you&#39;re limited in the HashMap implementations you can use. Personally, I<br>
&gt; don&#39;t think it&#39;s a memory leak, as GUID number in the system are finite and<br>
&gt; not too large. What do you think? Want to add it to your patch?<br>
&gt;<br>
&gt; On Jun 30, 2013, at 11:13 AM, Yair Zaslavsky wrote:<br>
&gt;<br>
&gt; &gt; Well done, should have been done ages ago :)<br>
&gt; &gt; Now, for the painful rebase of async_task_mgr changes :)<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; ----- Original Message -----<br>
&gt; &gt;&gt; From: &quot;Allon Mureinik&quot; &lt;<a href="mailto:amureini@redhat.com">amureini@redhat.com</a>&gt;<br>
&gt; &gt;&gt; To: &quot;engine-devel&quot; &lt;<a href="mailto:engine-devel@ovirt.org">engine-devel@ovirt.org</a>&gt;, &quot;Barak Azulay&quot;<br>
&gt; &gt;&gt; &lt;<a href="mailto:bazulay@redhat.com">bazulay@redhat.com</a>&gt;<br>
&gt; &gt;&gt; Cc: &quot;Yair Zaslavsky&quot; &lt;<a href="mailto:yzaslavs@redhat.com">yzaslavs@redhat.com</a>&gt;, &quot;Michael Pasternak&quot;<br>
&gt; &gt;&gt; &lt;<a href="mailto:mpastern@redhat.com">mpastern@redhat.com</a>&gt;, &quot;Tal Nisan&quot;<br>
&gt; &gt;&gt; &lt;<a href="mailto:tnisan@redhat.com">tnisan@redhat.com</a>&gt;, &quot;Ayal Baron&quot; &lt;<a href="mailto:abaron@redhat.com">abaron@redhat.com</a>&gt;<br>
&gt; &gt;&gt; Sent: Sunday, June 30, 2013 11:11:30 AM<br>
&gt; &gt;&gt; Subject: Guid improvements<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Hi all,<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; I just merged a couple of improvements to the [N]Guid class [1] to improve<br>
&gt; &gt;&gt; it&#39;s performance both CPU-wise and memory-wise, based on a set of<br>
&gt; &gt;&gt; benchmarks<br>
&gt; &gt;&gt; presented by Liran.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; What this patchset achieves:<br>
&gt; &gt;&gt; 1. Clean up the code, so it&#39;s easier to understand and use<br>
&gt; &gt;&gt; 2. Eliminate the inflation in the memory foot print caused by the<br>
&gt; &gt;&gt; getValue()<br>
&gt; &gt;&gt; method<br>
&gt; &gt;&gt; 3. Eliminate all the heavy calls to UUID.fromString when creating a<br>
&gt; &gt;&gt; new/empty<br>
&gt; &gt;&gt; Guid instance as a default value<br>
&gt; &gt;&gt; 4. Note that the cleanups proposed in (1) will have minor performance<br>
&gt; &gt;&gt; benefits (e.g., eliminating useless conditional statements), but I doubt<br>
&gt; &gt;&gt; this would be anything to write home about.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; From a developer&#39;s perspective, here&#39;s what changed:<br>
&gt; &gt;&gt; 1. No more NGuid, just Guid. Both static methods to create a Guid from<br>
&gt; &gt;&gt; String<br>
&gt; &gt;&gt; still exist, and are named createGuidFromString and<br>
&gt; &gt;&gt; createGuidFromStringDefaultEmpty.<br>
&gt; &gt;&gt; 2. [N]Guid.getValue() was removed, it&#39;s no longer needed after (1) was<br>
&gt; &gt;&gt; implemented<br>
&gt; &gt;&gt; 3. The Guid() constructor was made private, as it forced a redundant call<br>
&gt; &gt;&gt; to<br>
&gt; &gt;&gt; UUID.fromString(String). If you need an empty Guid instance, just use<br>
&gt; &gt;&gt; Guid.Empty<br>
&gt; &gt;&gt; 4. The Guid.EMPTY_GUID_VALUE string constant was removed, as it was used<br>
&gt; &gt;&gt; for<br>
&gt; &gt;&gt; redundant calls to UUID.fromString. If you really, REALLY, need it, just<br>
&gt; &gt;&gt; call Guid.Empty.getValue() for a UUID or Guid.Empty.toString() for a<br>
&gt; &gt;&gt; String.<br>
&gt; &gt;&gt; 5. All sorts of ways to transform Strings to Guids were removed. If you<br>
&gt; &gt;&gt; have<br>
&gt; &gt;&gt; a literal you trust, just use new Guid(String). If you suspect it may be<br>
&gt; &gt;&gt; null, use Guid.createGuidFromString[DefaultEmpty]<br>
&gt; &gt;&gt; 6. NewGuid is now called newGuid. We&#39;re in Java, not C# :-)<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Many thanks to everyone who reviewed this patchset.<br>
&gt; &gt;&gt; You guys rock!<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Regards,<br>
&gt; &gt;&gt; Allon<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; [1]<br>
&gt; &gt;&gt; <a href="http://gerrit.ovirt.org/#/q/project:ovirt-engine+branch:master+topic:guid-cleanup,n,z" target="_blank">http://gerrit.ovirt.org/#/q/project:ovirt-engine+branch:master+topic:guid-cleanup,n,z</a><br>
&gt; &gt;&gt;<br>
&gt; &gt; _______________________________________________<br>
&gt; &gt; Engine-devel mailing list<br>
&gt; &gt; <a href="mailto:Engine-devel@ovirt.org">Engine-devel@ovirt.org</a><br>
&gt; &gt; <a href="http://lists.ovirt.org/mailman/listinfo/engine-devel" target="_blank">http://lists.ovirt.org/mailman/listinfo/engine-devel</a><br>
&gt;<br>
&gt;<br>
</div></div></blockquote></div><br></div>