electricdusk/rushlink
3
3
Fork 3
Code Issues 21 Pull Requests 2 Releases Activity

Add request_duration_seconds metric #72

Merged
mrngm merged 3 commits from metrics into master 2021-05-16 21:10:49 +02:00
Conversation 1 Commits 3 Files Changed 2 +16 -13
1 changed files with 16 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 306705cb28 - Show all commits

19
metrics.go
View File

@ -87,19 +87,22 @@ func (mh *MetricsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
func (mh *MetricsHandler) updateMetrics() { func (mh *MetricsHandler) updateMetrics() {
// Update metricURLsTotalGauge // Update metricURLsTotalGauge
electricdusk marked this conversation as resolved Outdated
mrngm commented 2021-05-16 14:31:00 +02:00
Outdated
Review
Copy Link

Is it possible to query the database once, yielding all counts for all types and states?

Is it possible to query the database once, yielding all counts for all types and states?
electricdusk commented 2021-05-16 17:34:59 +02:00
Outdated
Review
Copy Link

I tried to find this, but thought it was not not possible. I have retried searching for this and found this: https://stackoverflow.com/a/19046871/5207081
That would fix this.

I tried to find this, but thought it was not not possible. I have retried searching for this and found this: https://stackoverflow.com/a/19046871/5207081 That would fix this.
mrngm commented 2021-05-16 18:29:02 +02:00
Outdated
Review
Copy Link

Ran this query on the test migration database:

sqlite> SELECT type, state, COUNT(*) FROM pastes GROUP BY type, state;
0|2|136  // type: undef      state: deleted
2|1|118  // type: redirect   state: present
3|1|1376 // type: fileupload state: present

Would that roughly be what you expect to get here?

Ran this query on the test migration database: ``` sqlite> SELECT type, state, COUNT(*) FROM pastes GROUP BY type, state; 0|2|136 // type: undef state: deleted 2|1|118 // type: redirect state: present 3|1|1376 // type: fileupload state: present ``` Would that roughly be what you expect to get here?
for state := db.PasteStateUndef; state <= db.PasteStateDeleted; state++ { results := make([](struct {
for ty := db.PasteTypeUndef; ty <= db.PasteTypeFileUpload; ty++ { Type db.PasteType
var count int64 State db.PasteState
query := mh.db.Unscoped().Model(&db.Paste{}).Where("type = ? AND state = ?", ty, state).Count(&count) Count float64
}), 0)
query := mh.db.Unscoped().Model(&db.Paste{}).Select("type", "state", "COUNT(*) as count").Group("type, state").Find(&results)
if err := query.Error; err != nil { if err := query.Error; err != nil {
log.Printf("error: %v", errors.Wrap(err, "fetching pastes_total metric")) log.Printf("error: %v", errors.Wrap(err, "fetching pastes_total metric"))
return return
} }
metricURLsTotalGauge.Reset()
for _, r := range results {
labels := map[string]string{ labels := map[string]string{
"state": state.String(), "type": r.Type.String(),
"type": ty.String(), "state": r.State.String(),
}
metricURLsTotalGauge.With(labels).Set(float64(count))
} }
metricURLsTotalGauge.With(labels).Set(r.Count)
} }
} }