Problem/Motivation

The created time was changed to use microtime, I think the setMultiple patch was done at a similar time. This still uses request time though. We need to keep parity.

Proposed resolution

Make sure we use microtime() in doSetMultiple() too. This is not an issue for most other cache backends, as setMultiple often just iterates and calls set(). E.g. ApcuBackend.

Remaining tasks

patch, small amount of test coverage

User interface changes

None

API changes

None

CommentFileSizeAuthor
d8.db_backend.setMultiple.microtime.patch1.66 KBdamiankloip
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed a4f2506 and pushed to 8.x. Thanks!

  • alexpott committed a4f2506 on 8.x 
    Issue #2298485 by damiankloip: Fixed setMultiple should also use...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.