Page MenuHomePhabricator

cul_reason in cu_log should use the comment table
Closed, ResolvedPublic

Description

The cu_log table stores the reason for the check directly in cul_reason. This duplication is likely larger than most other tables because checks often go in a chain where the reason is kept the same over many many checks. Looking at enwiki's CheckUserLog on the first page there is +30 checks with the same reason. This is very likely to lead to massive duplication and using the comment table is going to save on space needed.

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

Dreamy_Jazz renamed this task from cul_reason should be normalized by using the comment table to cul_reason in cu_log should use the comment table.May 29 2022, 11:30 AM

Don't think this should be marked as a "Bug report". If someone knows how to change that please do.

Aklapper changed the subtype of this task from "Bug Report" to "Task".May 29 2022, 11:56 AM

Change 800827 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Allow searching CheckUserLog by check reason with wildcards

https://gerrit.wikimedia.org/r/800827

Change 820835 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Start writing to the comment table for the cu_log reason

https://gerrit.wikimedia.org/r/820835

Change 820835 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Start writing to the comment table for the cu_log reason

https://gerrit.wikimedia.org/r/820835