Changes for page Notifications Optimization
Last modified by Vincent Massol on 2024/02/26 17:53
<
>
edited by Guillaume Delhumeau
on 2019/03/06 15:08
on 2019/03/06 15:08
edited by Guillaume Delhumeau
on 2019/03/06 15:09
on 2019/03/06 15:09
Change comment:
There is no comment for this version
Summary
-
Page properties (1 modified, 0 added, 0 removed)
Details
- Page properties
-
- Content
-
... ... @@ -32,8 +32,19 @@ 32 32 33 33 == Solution 1 - A: Add a timeout to the query == 34 34 35 -To avoid an infinite loop 35 +To avoid an infinite loop, we could simply add a timeout. 36 36 37 +Pros: 38 + 39 +* simple and efficient (see the commit https:~/~/github.com/xwiki/xwiki-platform/commit/bc59ddaaaa062ba3e946063c1e1b01597ad3d6b8 before it was reverted) 40 + 41 +((( 42 +Cons: 43 + 44 +* some important notifications could be lost 45 +* it does not really fix the issue. It just hides it. 46 +))) 47 + 37 37 = Problem 2: Notifications are computed each time a page is loaded = 38 38 39 39 There is absolutely no cache mechanism. So, even if the query to fetch the notifications is long, it will be re-executed the next time a user loads a page.