Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Telemetry Source Before/After events feedback/thoughts #3204

Closed
avanderhoorn opened this issue Sep 25, 2015 · 5 comments
Closed

Telemetry Source Before/After events feedback/thoughts #3204

avanderhoorn opened this issue Sep 25, 2015 · 5 comments
Assignees
Milestone

Comments

@avanderhoorn
Copy link
Member

Currently there are a few places where we have various Before/After events - BeforeAction/AfterAction and BeforeActionMethod/AfterActionMethod being amongst them. In both of these cases the Before events has more details than the After event. This means that for people interested in getting that before data in conjunction with the timing data have to subscribe to both events and either independently store the data from the begin event and carry it though to the after, or correlate the data after the fact.

In most cases, if the After event had the same data as the Before event, it would simplify things for the developer with the only thing they would need to store/flow between the two events is the timing information. In our case, it would also mean that we would only need to publish the After event to our internal store and/or not have to store this subset of data independently whilst waiting for the post event if we only wanted the one event going through to our internal store.

For the current Before/After events where there is a difference in parameters that are published, it would seem that the extra data the Before event has could easily be made available to the After event as well. And moving forward it would seem this pattern would continue as the After usual is executed in the same scope as the Before.

@danroth27
Copy link
Member

@rynowak

@rynowak
Copy link
Member

rynowak commented Oct 5, 2015

Discussed this a bit with @avanderhoorn this morning, he thinks the changes here are pretty minimal and is going to send a PR 👍

@Eilon
Copy link
Member

Eilon commented Oct 5, 2015

@rynowak parking this on your plate for tracking.

@rynowak rynowak added 3 - Done and removed 1 - Ready labels Oct 6, 2015
@rynowak
Copy link
Member

rynowak commented Oct 6, 2015

a0764fa

@rynowak rynowak closed this as completed Oct 6, 2015
@avanderhoorn
Copy link
Member Author

Thanks!

On Monday, October 5, 2015, Ryan Nowak [email protected] wrote:

Closed #3204 #3204.


Reply to this email directly or view it on GitHub
#3204 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants