HomePhabricator
Diffusion Comm 8266d4d872f1

[lib] Consider parent timestamp in lastUpdatedTimeIncludingSidebars

Description

[lib] Consider parent timestamp in lastUpdatedTimeIncludingSidebars

Summary:
I made a mistake in D14154: I forgot that we should always consider the parent thread's timestamp in lastUpdatedTimeIncludingSidebars.

Depends on D14156

Test Plan: Confirm that we no longer see a massive performance regression

Reviewers: varun, will

Reviewed By: varun

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D14157

Details

Provenance
ashoatAuthored on Fri, Dec 13, 2:52 PM
Reviewer
varun
Differential Revision
D14157: [lib] Consider parent timestamp in lastUpdatedTimeIncludingSidebars
Parents
rCOMM730c48dec5a5: [lib] Fix stack overflow from lastUpdatedTime
Branches
Unknown
Tags
Unknown