Revert change to counting of deactivated users towards the monthly active users limit (#11127)
Temporarily revert "Add functionality to remove deactivated users from the monthly_active_users table (#10947)".
This reverts commit eda8c88b84ee7506379a71ac2a7a88c08b759d43.
2 files changed, 0 insertions, 28 deletions
diff --git a/synapse/handlers/deactivate_account.py b/synapse/handlers/deactivate_account.py
index e88c3c27ce..bee62cf360 100644
--- a/synapse/handlers/deactivate_account.py
+++ b/synapse/handlers/deactivate_account.py
@@ -131,10 +131,6 @@ class DeactivateAccountHandler:
# delete from user directory
await self.user_directory_handler.handle_local_user_deactivated(user_id)
- # If the user is present in the monthly active users table
- # remove them
- await self.store.remove_deactivated_user_from_mau_table(user_id)
-
# Mark the user as erased, if they asked for that
if erase_data:
user = UserID.from_string(user_id)
diff --git a/synapse/storage/databases/main/monthly_active_users.py b/synapse/storage/databases/main/monthly_active_users.py
index ec4d47a560..a14ac03d4b 100644
--- a/synapse/storage/databases/main/monthly_active_users.py
+++ b/synapse/storage/databases/main/monthly_active_users.py
@@ -354,27 +354,3 @@ class MonthlyActiveUsersStore(MonthlyActiveUsersWorkerStore):
await self.upsert_monthly_active_user(user_id)
elif now - last_seen_timestamp > LAST_SEEN_GRANULARITY:
await self.upsert_monthly_active_user(user_id)
-
- async def remove_deactivated_user_from_mau_table(self, user_id: str) -> None:
- """
- Removes a deactivated user from the monthly active user
- table and resets affected caches.
-
- Args:
- user_id(str): the user_id to remove
- """
-
- rows_deleted = await self.db_pool.simple_delete(
- table="monthly_active_users",
- keyvalues={"user_id": user_id},
- desc="simple_delete",
- )
-
- if rows_deleted != 0:
- await self.invalidate_cache_and_stream(
- "user_last_seen_monthly_active", (user_id,)
- )
- await self.invalidate_cache_and_stream("get_monthly_active_count", ())
- await self.invalidate_cache_and_stream(
- "get_monthly_active_count_by_service", ()
- )
|