From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: use *c*ElementTree
......................................................................


hostdev: use *c*ElementTree

cElementTree is *the* recommended ETree implementation on platforms
that support C. It delivers nice performance improvement over previous
implementation.

ElementTree:

time PYTHONPATH="../" NOSE_SLOW_TESTS=true ./run_tests.sh -s -m
test_3k_storage_devices hostdevTests.py
nose.config: INFO: Ignoring files matching ['^\\.', '^_',
'^setup\\.py$']
hostdevTests.HostdevPerformanceTests
    test_3k_storage_devices                                     OK

----------------------------------------------------------------------
Ran 1 test in 2.656s

OK

real    0m3.010s
user    0m2.951s
sys     0m0.060s

cElementTree:

time PYTHONPATH="../" NOSE_SLOW_TESTS=true ./run_tests.sh -s -m
test_3k_storage_devices hostdevTests.py
nose.config: INFO: Ignoring files matching ['^\\.', '^_',
'^setup\\.py$']
hostdevTests.HostdevPerformanceTests
    test_3k_storage_devices                                     OK

----------------------------------------------------------------------
Ran 1 test in 0.432s

OK

real    0m0.789s
user    0m0.730s
sys     0m0.060s

Change-Id: I580e3f1c30c4751f0299b6fbc2bd5eb6ed33132b
Bug-Url: https://bugzilla.redhat.com/1398572
Signed-off-by: Martin Polednik <mpoled...@redhat.com>
---
M lib/vdsm/hostdev.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Martin Polednik: Verified



-- 
To view, visit https://gerrit.ovirt.org/67515
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I580e3f1c30c4751f0299b6fbc2bd5eb6ed33132b
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpoled...@redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to