Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#1593 closed task (wontfix)

deprecate midcom_services_cache_backend::open,close,exists

Reported by: rambo Owned by: indeyets
Priority: major Milestone: 8.09.8 Ragnaroek
Component: MidCOM component Version: 8.09 Ragnaroek
Keywords: cache Cc:

Description

This means also deprecating some backends that depend on them, or at least reworking them to do without.

the exists check is largely unneccessary, basically it's only used because some backends throw hissy fit if you try to read a key that does not exist, some other backends have actually no way to check if key exists (other than reading it), notably memcached, and for them use of the method is completely useless.

Same for open/close, they're only needed for backends where one cannot do read and write at the same time (or writing is so expensive, think flatfile, that writes need to be batched).

Change History (4)

comment:1 Changed 7 years ago by rambo

#1048 (delaying cache writes) basically requires that this is done as well.

comment:2 Changed 7 years ago by bergie

  • Keywords cache added
  • Owner changed from bergie to indeyets

Bugs related to caching in MidCOM:

  • #1048 - Save things to cache in the end of request
  • #1593 - deprecate midcom_services_cache_backend::open,close,exists
  • #1599 - Prevent unwanted cache writes

comment:3 Changed 7 years ago by indeyets

  • Resolution set to wontfix
  • Status changed from new to closed

this is not a requirement for fixing #1048 and, actually, it gives a bit of flexibility. leaving as is

comment:4 Changed 7 years ago by flack

(In [26360]) skip empty open/close calls in memcache backend, refs #1593

Note: See TracTickets for help on using tickets.