Cache rules everything around me.

This is sorta important because if you want to use apachesolr and memcached, the wildcard clearing causes a general cache wipe, and thus a lot of expensive rebuilds.

Comments

robertdouglass’s picture

StatusFileSize
new2.98 KB

Just for fun, compare the patch that I made to do the same this morning :)

JacobSingh’s picture

Component: SolrPHP Client » Code

$$bill ya'll :P

I think this is a good commit any objections?

robertdouglass’s picture

Version: 6.x-2.x-dev » 5.x-2.x-dev
Status: Needs review » Patch (to be ported)
StatusFileSize
new1.65 KB

Committed to 6.1 and 6.2.

Claudiu, you'll need to edit .install so that it creates the table and does the update.

joshk’s picture

@RobertDouglas: description and default serialization FTW!

pwolanin’s picture

StatusFileSize
new2 KB

I don't think we need to set the serialized value if we are using cache_set() - in fact, doing so seems not to have any effect based on the cache_set() code.

Committing this minor follow-up also to the 6 branches.

pwolanin’s picture

Version: 5.x-2.x-dev » 6.x-1.x-dev
Status: Patch (to be ported) » Needs work

argh.

Robert's patch missed changing the calls to cache_get() ...

pwolanin’s picture

Priority: Normal » Critical
Status: Needs work » Needs review
StatusFileSize
new1.72 KB

Follow-up/clean-up

robertdouglass’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Priority: Critical » Normal
Status: Needs review » Needs work
StatusFileSize
new1.31 KB

Yup, two of them.

robertdouglass’s picture

Version: 6.x-2.x-dev » 5.x-2.x-dev
Status: Needs work » Patch (to be ported)

#630798 by robertDouglass, pwolanin, joshk | JacobSingh: Added Implement cache_apachesolr() table.

Fixed in both 6.x branches.

claudiu.cristea’s picture

StatusFileSize
new5.4 KB

Here's the patch for DRUPAL-5--2...

I have only one comment: "apachesolr:" should be stripped out from CIDs while we have our own table for caching... Right?

claudiu.cristea’s picture

StatusFileSize
new6.21 KB

Forgot the CHANGELOG.txt :)

claudiu.cristea’s picture

Status: Patch (to be ported) » Fixed

Committed in #317530.

Status: Fixed » Closed (fixed)

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