Change Details

PageTriageUtil::getTopTriagers() currently relies on data from the pagetriage_log table. However this data is not reliable since pagetriage_log only includes data for pages that are still in the PageTriage queue (either because they are unreviewed or because they haven't been garbage collected yet). We need to either switch PageTriageUtil::getTopTriagers() to look for data in the main logging table or we need to make pagetriage_log into a permanent log (that isn't synced with pagetriage_page). (See updatePageTriageQueue.php and PageTriage::deleteFromPageTriage() for the code that keeps them in sync.) I'm not really sure what the purpose of the pagetriage_log table actually is, as it seems to be largely redundant with the same info in the logging table.

PageTriageUtil::getTopTriagers() currently relies on data from the pagetriage_log table. However this data is not reliable since pagetriage_log only includes data for pages that are still in the PageTriage system (either because they are unreviewed or because they haven't been garbage collected yet). We need to either switch PageTriageUtil::getTopTriagers() to look for data in the main logging table or we need to make pagetriage_log into a permanent log (that isn't synced with pagetriage_page). (See updatePageTriageQueue.php and PageTriage::deleteFromPageTriage() for the code that keeps them in sync.) I'm not really sure what the purpose of the pagetriage_log table actually is, as it seems to be largely redundant with the same info in the logging table.

PageTriageUtil::getTopTriagers() currently relies on data from the pagetriage_log table. However this data is not reliable since pagetriage_log only includes data for pages that are still in the PageTriage queuesystem (either because they are unreviewed or because they haven't been garbage collected yet). We need to either switch PageTriageUtil::getTopTriagers() to look for data in the main logging table or we need to make pagetriage_log into a permanent log (that isn't synced with pagetriage_page). (See updatePageTriageQueue.php and PageTriage::deleteFromPageTriage() for the code that keeps them in sync.) I'm not really sure what the purpose of the pagetriage_log table actually is, as it seems to be largely redundant with the same info in the logging table.