-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Metrics #24
Comments
We can do this easiest by putting a
That's the same thing, I think. Add a
We have this, the
This is also data we have (or absence of it) |
Why should we use InfuxDB? It's fun to play with new tech and all, but adds a lot of overhead.
This data we already have, can make a script to extract it into CSV at some point. |
Does this ticket contain anything actionable that you want to do now? |
ive used influxdb for metrics tracking in other projects, and would continue to use it with a dashboard like there is nothing im going to do today but i'd it is an important technical story to have metrics once core functionality is done |
Ok. About adding another DB I'm wary - with risk of sounding boring, databases are not our secret sauce. |
I get it, and if we have to choose just one my vote is on Postgresql. FWIW influxdb has an excellent query DSL for time related data (which user metrics and app metrics are) and are still something I'd like to do |
So, the changes we need to implement to satisfy the bullets in your "Metrics" list are these:
|
Working on this right now... |
metrics:
Overall
The text was updated successfully, but these errors were encountered: