cmd/ursrv: Filter out ancient versions from chart

This commit is contained in:
Jakob Borg 2020-04-16 09:13:01 +02:00
parent 41ef945b2b
commit 5373e38ac8
2 changed files with 25 additions and 5 deletions

View File

@ -23,6 +23,7 @@ import (
"os" "os"
"regexp" "regexp"
"sort" "sort"
"strconv"
"strings" "strings"
"sync" "sync"
"time" "time"
@ -900,7 +901,8 @@ func newDataHandler(db *sql.DB, w http.ResponseWriter, r *http.Request) {
} }
func summaryHandler(db *sql.DB, w http.ResponseWriter, r *http.Request) { func summaryHandler(db *sql.DB, w http.ResponseWriter, r *http.Request) {
s, err := getSummary(db) min, _ := strconv.Atoi(r.URL.Query().Get("min"))
s, err := getSummary(db, min)
if err != nil { if err != nil {
log.Println("summaryHandler:", err) log.Println("summaryHandler:", err)
http.Error(w, "Database Error", http.StatusInternalServerError) http.Error(w, "Database Error", http.StatusInternalServerError)
@ -1558,7 +1560,21 @@ func (s *summary) MarshalJSON() ([]byte, error) {
return json.Marshal(table) return json.Marshal(table)
} }
func getSummary(db *sql.DB) (summary, error) { // filter removes versions that never reach the specified min count.
func (s *summary) filter(min int) {
// We cheat and just remove the versions from the "index" and leave the
// data points alone. The version index is used to build the table when
// we do the serialization, so at that point the data points are
// filtered out as well.
for ver := range s.versions {
if s.max[ver] < min {
delete(s.versions, ver)
delete(s.max, ver)
}
}
}
func getSummary(db *sql.DB, min int) (summary, error) {
s := newSummary() s := newSummary()
rows, err := db.Query(`SELECT Day, Version, Count FROM VersionSummary WHERE Day > now() - '2 year'::INTERVAL;`) rows, err := db.Query(`SELECT Day, Version, Count FROM VersionSummary WHERE Day > now() - '2 year'::INTERVAL;`)
@ -1589,6 +1605,7 @@ func getSummary(db *sql.DB) (summary, error) {
s.setCount(day.Format("2006-01-02"), ver, num) s.setCount(day.Format("2006-01-02"), ver, num)
} }
s.filter(min)
return s, nil return s, nil
} }

View File

@ -55,7 +55,10 @@ found in the LICENSE file.
google.setOnLoadCallback(drawPerformanceCharts); google.setOnLoadCallback(drawPerformanceCharts);
function drawVersionChart() { function drawVersionChart() {
var jsonData = $.ajax({url: "summary.json", dataType:"json", async: false}).responseText; // Summary version chart for versions that at some point in the chart
// reaches 250 devices. This filters out versions that are old and
// uninteresting yet linger forever with like four users.
var jsonData = $.ajax({url: "summary.json?min=250", dataType:"json", async: false}).responseText;
var rows = JSON.parse(jsonData); var rows = JSON.parse(jsonData);
var data = new google.visualization.DataTable(); var data = new google.visualization.DataTable();