Skip to content

Instrument RQ#375

Merged
adamchainz merged 7 commits intomasterfrom
issue_370_instrument_rq
Nov 21, 2019
Merged

Instrument RQ#375
adamchainz merged 7 commits intomasterfrom
issue_370_instrument_rq

Conversation

@adamchainz
Copy link
Copy Markdown
Contributor

Fixes #370.

@adamchainz adamchainz changed the title Instrument RQ [WIP] Instrument RQ Nov 12, 2019
@adamchainz adamchainz changed the title [WIP] Instrument RQ Instrument RQ Nov 15, 2019
@adamchainz adamchainz force-pushed the issue_370_instrument_rq branch 2 times, most recently from 1b162ed to 674acdc Compare November 15, 2019 16:08
@adamchainz
Copy link
Copy Markdown
Contributor Author

Want to write the docs before I merge this. Had to consider a few designs, eventually settled on a custom Worker class with decorator-patching of Job that seems to do everything we'd want.

@adamchainz
Copy link
Copy Markdown
Contributor Author

Docs are up at scoutapp/slate_apm_help#70 . I've retested my rq test apps and am happy with this now!

@adamchainz adamchainz merged commit cd1c447 into master Nov 21, 2019
@adamchainz adamchainz deleted the issue_370_instrument_rq branch November 21, 2019 14:34
adamchainz added a commit that referenced this pull request Nov 25, 2019
adamchainz added a commit that referenced this pull request Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Instrument rq

1 participant