Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Add KPI Metrics logging statement #448

Closed
bbangert opened this issue Apr 20, 2016 · 0 comments
Closed

Add KPI Metrics logging statement #448

bbangert opened this issue Apr 20, 2016 · 0 comments

Comments

@bbangert
Copy link
Member

bbangert commented Apr 20, 2016

A new metric statement should be added when the client drops its connection, that logs some session statistics for the user-session.

Example logging message:

self.log.info("Session", connection_type="websocket", messages=3, register=0,
              unregister=0, new_uaid=False, uaid_hash="...", user-agent="..."
              )

This is needed for the following KPI's:

  1. Daily Activations - Knowing new_uaid for a session indicates if the uaid is a reconnect or new.
  2. DIU’s - Daily Inactive Users, to know one, I'd need to see that for all the users sessions for a day, there were no messages.

It's possible to get both of these by alternatively logging when a new uaid is issued, and logging 'hello' for a uaid_hash, which would let me group them to see there were no Ack's. That's likely just as efficient, but correlating commands issued per 'session', and other per-session stats would be hard to evaluate. As such, it'd be easier to do analysis on a Session logging statement that had it all.

A side-effect of this is that long-lived connections get a bit.... weird. This shouldn't be that bad for our stats, as the max typical life span of a connection right now is around 50 minutes.

@bbangert bbangert added this to the PUSHSVC-0: quality milestone Apr 20, 2016
@bbangert bbangert added the ready label May 9, 2016
@bbangert bbangert self-assigned this Apr 12, 2017
@bbangert bbangert added in progress and removed ready labels May 3, 2017
bbangert added a commit that referenced this issue May 9, 2017
Logs a summary of session statistics when a client drops its client
connection. Also cleans up several places where the PushState had
its uaid set but not its uaid_obj or uaid_hash.

Closes #448
bbangert added a commit that referenced this issue May 9, 2017
Logs a summary of session statistics when a client drops its client
connection. Also cleans up several places where the PushState had
its uaid set but not its uaid_obj or uaid_hash.

Closes #448
bbangert added a commit that referenced this issue May 9, 2017
Logs a summary of session statistics when a client drops its client
connection. Also cleans up several places where the PushState had
its uaid set but not its uaid_obj or uaid_hash.

Closes #448
bbangert added a commit that referenced this issue May 9, 2017
Logs a summary of session statistics when a client drops its client
connection. Also cleans up several places where the PushState had
its uaid set but not its uaid_obj or uaid_hash.

Closes #448
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

2 participants