using an incremented counter, instead of pushing an inacurrate timing
data (if calling the encapsulated function raises an exception).
Depends on D1296
Details
Details
- Reviewers
olasd - Group Reviewers
Reviewers - Commits
- rDCOREae19bd9f88b0: statsd: make TimedContextManagerDecorator report errors
Diff Detail
Diff Detail
- Repository
- rDCORE Foundations and core functionalities
- Branch
- current
- Lint
Lint Skipped - Unit
Unit Tests Skipped - Build Status
Buildable 4845 Build 6459: tox-on-jenkins Jenkins Build 6458: arc lint + arc unit
Event Timeline
Comment Actions
Build has FAILED
Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/195/
See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/195/console
| swh/core/statsd.py | ||
|---|---|---|
| 108–110 | You can drop the start = None and replace finally:
if start is not None:
self._send(start)by: else:
self._send(start)(same comment for the block below) | |
| swh/core/statsd.py | ||
|---|---|---|
| 108–110 | gheez, you are damn right! | |
| swh/core/statsd.py | ||
|---|---|---|
| 108–110 | In fact no, at least not exactly, because of the return statement in the try block. | |
| swh/core/statsd.py | ||
|---|---|---|
| 108–110 | Oops indeed. The return could be at the end of the function, though. | |
Comment Actions
Build has FAILED
Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/197/
See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/197/console
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DCORE/job/tox/198/ for more details.