[Freeipa-devel] [PATCH] test speedup patch

[JR Aquino]

Just tested.  I do see a performance increase of ~30%

<Without the Patch>
time ipa user-find
--------------
1 user matched
--------------
  User login: admin
  Last name: Administrator
  Home directory: /home/admin
  Login shell: /bin/bash
  Account activation status: False
  Member of groups: admins
----------------------------
Number of entries returned 1
----------------------------

real 0m1.558s
user 0m0.810s
sys 0m0.165s


<With the Patch>
time ipa user-find
--------------
1 user matched
--------------
  User login: admin
  Last name: Administrator
  Home directory: /home/admin
  Login shell: /bin/bash
  Account disabled: False
  Member of groups: admins
----------------------------
Number of entries returned 1
----------------------------

real 0m1.067s
user 0m0.841s
sys 0m0.157s



On 1/18/11 7:17 PM, "Rob Crittenden" <rcritten at redhat.com> wrote:

>Rob Crittenden wrote:
>> Rob Crittenden wrote:
>>> Attached is a rough cut of a patch to try to speed up the cli a little
>>> bit. Basically in production mode it will skip some things during
>>> initialization.
>>>
>>> My concept is that we develop in mode != production and release in mode
>>> == production.
>>>
>>> I managed to knock a second or so off time to do a user-show on
>>>average.
>>>
>>> There may be some other things we can do to speed things up, I'm still
>>> looking. Some feedback on the approach would be appreciated.
>>>
>>> Note that I've completely ruled out SSL/Negotiate. I did my testing on
>>> lite-server which doesn't use SSL or Negotiate and it was STILL taking
>>> on average 3-4+ seconds per command. The server side was consistently
>>> taking < 1 second to complete.
>>>
>>> rob
>>
>> oh, and the patch.
>
>I ran a couple of moderate tests this evening that executed 42 separate
>operations like add, delete, and managing group membership. I ran this
>10 times each on 2 identical VMs, one with a bit older code and one with
>this patch then averaged the times.
>
>With the patch the average was 1.3 seconds per operation, without 2.6. A
>50% improvement is more than I expected, I saw a 33% improvement on
>individual runs. I'll keep at it but this seems promising. I was also a
>bit surprised that the average time without the patch was so low, I was
>expecting something over 3 seconds.
>
>Specifically what this patch does is it avoids doing some
>self-validation. There is some amount of risk that the framework could
>blow up but in a deployed situation I think the risk is rather low.
>
>A side-effect of the API tester makeapi is that it loads the framework.
>We can force it to be run in production mode so the product shouldn't be
>buildable if it has inconsistencies.
>
>rob
>
>_______________________________________________
>Freeipa-devel mailing list
>Freeipa-devel at redhat.com
>https://www.redhat.com/mailman/listinfo/freeipa-devel