Clean database cache from clear cache command #18119
Conversation
There was a problem hiding this comment.
Honestly, I do not think we should consider that clearing the ancestors_cache/sons_cache is a normal cache clearing operation, for 2 distinct reasons.
-
This cache contains only a list of IDs, so its contents is not supposed to changed and so there is not reason for it to become stale if the application invalidates/recompute it when it is required.
-
This cache is used, for entities, to determine what can or cannot be seen by users, and therefore it must never be stale. If it cannot be stale, then there is no reason to invalidate it.
If we know that a specific operation can result in an invalid value, then we have to fix it, as we do in #17012.
If we consider that we had in the past some bugs that can explain that the value of these fields is corrupted (for instance #17012), then maybe the solution is to invalidate them once in the 10.0.16 -> 10.0.17 update.
Description
Clean database cache from clear cache command