Skip to content

DB: Query total event count by period

Warren Gifford requested to merge event-logs-queries into master

Created by: efritz

This functionality will be necessary to query the metrics required by RFC 100 once all parties have approved.

This refactors the event log definitions in the db package so that we can add a new type of query that sums events rather than unique users in a time period.

This PR:

  • renames occurrences of "uniques" to "uniqueUser" to disambiguate what we're querying
  • adds conditions to filter results of CountUniqueUsersPerPeriod by event name/prefix
  • adds a definition of CountEventsPerPeriod that works similarly to CountUniqueUsersPerPeriod.

Review by commit may be easiest.

This PR defines but does not yet use this new method.

Merge request reports