Remove container list morph cache #806

New issue

Closed

opened 2023-11-15 10:04:45 +00:00 by fyrchik · 0 comments

fyrchik commented

2023-11-15 10:04:45 +00:00

Owner

Get/DeletionInfo cache has multiple usecases:

Policer iterates over containers sequentially
User load in the same container fetches placement policy

I couldn't come up with similar usecases for List:

Tree service uses list, but for all containers, not for a specific owner. This is not cached (and we don't need, see 072a7d61ab)
Polling for batch removal (control ir remove-container) is a possible usecase, but too narrow anyway.

The sugestion is to remove this cache.

Get/DeletionInfo cache has multiple usecases: 1. Policer iterates over containers sequentially 2. User load in the same container fetches placement policy I couldn't come up with similar usecases for List: 1. Tree service uses list, but for _all_ containers, not for a specific owner. This is not cached (and we don't _need_, see 072a7d61ab11b23e) 2. Polling for batch removal (`control ir remove-container`) is a possible usecase, but too narrow anyway. The sugestion is to remove this cache.