Page MenuHomePhabricator

Use the new comment table for summaries
Closed, ResolvedPublic

Description

rev_comment, log_comment, etc. are now deprecated. Comments live in the comment table. Everything at https://www.mediawiki.org/wiki/Manual:Database_layout is up-to-date.

This task depends on the comments being backfilled as part of T166733: Deploy refactored comment storage

Event Timeline

MusikAnimal renamed this task from Update to use the new comment table for summaries to Use the new comment table for summaries.Mar 8 2018, 7:10 PM
MusikAnimal claimed this task.
MusikAnimal moved this task from Backlog to Working on the XTools board.
MusikAnimal moved this task from Working to Blocked (Non-XTools) on the XTools board.

Err, the replicas aren't using it yet. Putting on hold.

T166733#4694462
time to update all our tools and scripts
:(

Access to the raw (okay, filtered) DBs has some advantages but it does mean you don't get a stable API etc.

XTools can't feasibly use the API. But anyway I'm seeing some missing historical data:

MariaDB [enwiki_p]> SELECT rev_comment, rev_comment_id FROM revision WHERE rev_id = 523232421\G;
*************************** 1. row ***************************
   rev_comment: [[WP:AES|]]Created page with '[[Category:Professional wrestling championships by promotion]] [[Category:Women's professional wrestling championships]]  ja:Category:NEO女子プロレスの...'
rev_comment_id: 0
1 row in set (0.12 sec)

where the rev_comment_id is 0

Access to the raw (okay, filtered) DBs has some advantages but it does mean you don't get a stable API etc.

Of course, I understand changes are needed.

MusikAnimal changed the task status from Open to Stalled.Oct 30 2018, 4:18 PM
MusikAnimal triaged this task as High priority.
MusikAnimal updated the task description. (Show Details)
MusikAnimal changed the task status from Stalled to Open.Nov 1 2018, 2:24 AM

Temporary fix has been added to check both revision/logging and comment. Leaving this in the Blocked column while we wait for confirmation that comments have been fully backfilled, at which time we'll remove the querying for rev_comment/log_comment entirely.

MusikAnimal moved this task from Pending deployment to Complete on the XTools board.