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.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Start writing to the comment table for the cu_log reason | mediawiki/extensions/CheckUser | master | +142 -3 |
| Status | Subtype | Assigned | Task | |
|---|---|---|---|---|
| · · · | ||||
| Resolved | Zabe | T233004 Update CheckUser for actor and comment table | ||
| Resolved | Dreamy_Jazz | T309454 cul_reason in cu_log should use the comment table | ||
| Declined | None | T309982 Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_actor to the cu_log table | ||
| · · · |
Event Timeline
Comment Actions
Don't think this should be marked as a "Bug report". If someone knows how to change that please do.
Comment Actions
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
Comment Actions
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
Comment Actions
Change 820835 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Start writing to the comment table for the cu_log reason