Page MenuHomePhabricator

Read views HTML and Edit views HTML for a page should come from the same ParserCache entry
Closed, ResolvedPublic

Description

Due to the different project paths (RESTBase Sunsetting, Parsoid Read Views), right now, we have two different code paths for generating HTML for read views and edit views. The read views code path is currently not deployed and when deployed, will initially only be used for testing.

Read views use on wikis needs to be blocked on ensuring that the ParserCache entries are shared between read and edit views to avoid unnecessary cache fragmentation.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
ssastry triaged this task as High priority.Apr 6 2023, 2:24 PM
ssastry moved this task from Needs Triage to Missing Functionality on the Parsoid board.

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

[mediawiki/core@master] Reconcile Parsoid opts in ParsoidOutputAccess & ParserOutputAccess

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

Change 946628 merged by jenkins-bot:

[mediawiki/core@master] Reconcile Parsoid opts in ParsoidOutputAccess & ParserOutputAccess

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

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

[mediawiki/core@master] WIP: Make ParsoidOutputAccess a wrapper over ParserOutputAccess

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

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

[mediawiki/core@master] WIP: Hacks to avoid cold cache misses after ParsoidOutputAccess changes

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

Change 961491 merged by jenkins-bot:

[mediawiki/core@master] Hacks to avoid cold cache misses after ParsoidOutputAccess changes

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

Change 953342 merged by jenkins-bot:

[mediawiki/core@master] Make ParsoidOutputAccess a wrapper over ParserOutputAccess

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

Nothing else to do here.

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

[mediawiki/core@REL1_41] Make ParsoidOutputAccess a wrapper over ParserOutputAccess

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

Change 969771 merged by jenkins-bot:

[mediawiki/core@REL1_41] Make ParsoidOutputAccess a wrapper over ParserOutputAccess

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