This module does a cache_clear_all() during its nodeapi(insert|update) operations. What is the benefit of this? These calls cost performance.

Comments

febbraro’s picture

febbraro commented
Assigned: Unassigned » febbraro

Not sure what that is doing there. This is a hold over from some older implementation (if I had to guess). We'll get this figured out and in the next release. Thanks for the review.

irakli’s picture

irakli commented
Assigned: febbraro » irakli
Status: Active » Fixed

That code looks like me being paranoid and putting extra cache invalidation. I think Moshe is right and it is not needed. Fixed in CVS.

Thank you.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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