dd-trace-py icon indicating copy to clipboard operation
dd-trace-py copied to clipboard

chore(debugging): clean up snapshot duration field

Open P403n1x87 opened this issue 3 years ago • 0 comments

Description

The semantics of the duration field was ambiguous and not aligned with other runtimes. This change cleans up the data model and ensures that the definition conforms to the new standard, which is that of using the duration field only as a subfield of the snapshot structure, bearing the value of the duration of the wrapped function.

Checklist

Reviewer Checklist

  • [ ] Title is accurate.
  • [ ] Description motivates each change.
  • [ ] No unnecessary changes were introduced in this PR.
  • [ ] PR cannot be broken up into smaller PRs.
  • [ ] Avoid breaking API changes unless absolutely necessary.
  • [ ] Tests provided or description of manual testing performed is included in the code or PR.
  • [x] Release note has been added for fixes and features, or else changelog/no-changelog label added.
  • [ ] All relevant GitHub issues are correctly linked.
  • [ ] Backports are identified and tagged with Mergifyio.
  • [ ] Add to milestone.

P403n1x87 avatar Aug 05 '22 10:08 P403n1x87