integrations-core icon indicating copy to clipboard operation
integrations-core copied to clipboard

DBM-1269: Track method execution statistics

Open alexbarksdale opened this issue 3 years ago • 2 comments

What does this PR do?

Decorates mysql methods to track performance with the new tracked_method decorator.

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

  • [ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • [ ] PR title must be written as a CHANGELOG entry (see why)
  • [ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • [ ] PR must have changelog/ and integration/ labels attached

alexbarksdale avatar May 18 '22 17:05 alexbarksdale

Codecov Report

Merging #12048 (8a8d970) into master (6e7254a) will decrease coverage by 1.94%. The diff coverage is 100.00%.

:exclamation: Current head 8a8d970 differs from pull request most recent head d1f220d. Consider uploading reports for the commit d1f220d to get more accurate results

Flag Coverage Δ
argocd ?
boundary ?
cert_manager ?
cloudera ?
ddev ?
ibm_ace ?
impala ?
mysql 89.00% <100.00%> (+1.64%) :arrow_up:
teradata ?

Flags with carried forward coverage won't be shown. Click here to find out more.

codecov[bot] avatar May 18 '22 18:05 codecov[bot]

Hi @alexbarksdale 👋 Sorry for the delay but do we want this to be released with 7.40?

FlorentClarret avatar Sep 14 '22 11:09 FlorentClarret

Closing this PR since it's pretty old, feel free to re-open it if needed 🙂

FlorentClarret avatar Jan 03 '24 08:01 FlorentClarret