diff options
| author |  Chris Lovering <[email protected]> | 2021-12-14 18:59:49 +0000 |
|---|---|---|
| committer | Chris Lovering <[email protected]> | 2021-12-14 18:59:49 +0000 |
| commit | f65184b5e454437c3c524d1236041f170fff3ea4 (patch) | |
| tree | 7b661a9954ae335f44fdfe12aa089cbb7e84ec1d | |
| parent | Merge pull request #634 from python-discord/bump-drf (diff) | |
Query message count directly from messages
This was changed due to performance reasons, but after some tweaking in the database, such as increasing work memory and adding an index, this query runs much faster now.
To test this, I want to revert this change, so that we can stop the materialised view from refreshing, to see if the act of refreshing is what's causing this query to seem faster when runing against the database.
| -rw-r--r-- | pydis_site/apps/api/models/bot/metricity.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/pydis_site/apps/api/models/bot/metricity.py b/pydis_site/apps/api/models/bot/metricity.py index 52e946ac..901f191a 100644 --- a/pydis_site/apps/api/models/bot/metricity.py +++ b/pydis_site/apps/api/models/bot/metricity.py @@ -46,12 +46,14 @@ class Metricity: self.cursor.execute( """ SELECT - message_count - FROM user_has_approx_message_count + COUNT(*) + FROM messages WHERE author_id = '%s' + AND NOT is_deleted + AND channel_id NOT IN %s """, - [user_id] + [user_id, EXCLUDE_CHANNELS] ) values = self.cursor.fetchone() |