Change in vdsm[master]: profile: Support yappi builtins option

danken at redhat.com danken at redhat.com
Tue Jun 17 08:36:50 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: profile: Support yappi builtins option
......................................................................


profile: Support yappi builtins option

yappi support profiling also builtin python functions. This patch adds a
new configuration value to enable profiling builtins in an application
wide profile, and a new builtins kwarg to the profile decorator.

This option can be handy when trying to understand why code is
expensive.

Change-Id: I67c6fc52ac64f8ff7f13471a53129a66361192ca
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/27787
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Saggi Mizrahi <smizrahi at redhat.com>
---
M lib/vdsm/config.py.in
M lib/vdsm/profile.py
M tests/profileTests.py
3 files changed, 70 insertions(+), 9 deletions(-)

Approvals:
  Nir Soffer: Verified
  Saggi Mizrahi: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I67c6fc52ac64f8ff7f13471a53129a66361192ca
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list