Page MenuHomePhabricator

Purge all old html version code from Parsoid repo once we switch VE from RESTBase to ParserCache
Open, LowPublic

Description

RESTBase is a store, not a cache. As such, there was on reliable way of knowing when old HTML versions would clear out from it and we also didn't have a mechanism to purge old versions. As such, Parsoid's codebase has a number of FIXMEs / TODOs to get rid of some code when the HTML for a version turns over in RESTBase.

But, we are now moving away from RESTBase to ParserCache where all entries turn over after X days (currently X = 21). So, once we fully switch over all wkies to ParserCache, after that X-day period, we should be able to get rid of all the b/c code in Parsoid that exists to deal with older HTML versions.

Event Timeline

daniel triaged this task as Low priority.Jun 5 2023, 6:17 PM
daniel moved this task from Unsorted to Parsoid pile on the RESTBase Sunsetting board.

Change 930649 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/services/parsoid@master] WIP: Purge b/c compatibility code for old HTML versions

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

Change 930649 abandoned by Subramanya Sastry:

[mediawiki/services/parsoid@master] WIP: Purge b/c compatibility code for old HTML versions

Reason:

Arlo pointed out that this is also blocked on https://phabricator.wikimedia.org/T174372 .. our dear friend Flow!

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