Skip to content

Allow tracking of the request's life cycle#238

Merged
jpd236 merged 4 commits intogoogle:masterfrom
artikz:request-tracking
Nov 20, 2018
Merged

Allow tracking of the request's life cycle#238
jpd236 merged 4 commits intogoogle:masterfrom
artikz:request-tracking

Conversation

@artikz
Copy link
Copy Markdown
Contributor

@artikz artikz commented Oct 31, 2018

Add request events listener which will be called on the request life
cycle events (queued, processing started, etc). Also add the ability to
associate data with the requests, without using other data structures,
which is useful when processing request life cycle events.

Add request events listener which will be called on the request life
cycle events (queued, processing started, etc). Also add the ability to
associate data with the requests, without using other data structures,
which is useful when processing request life cycle events.
@jpd236 jpd236 requested a review from jjoslin November 9, 2018 16:33
@artikz
Copy link
Copy Markdown
Contributor Author

artikz commented Nov 15, 2018

I think I addressed all the comments

Copy link
Copy Markdown
Collaborator

@jpd236 jpd236 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

generally seems okay to me. @jjoslin any further concerns on your side?

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.

3 participants