Closed (fixed)
Project:
Tagadelic
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
20 Mar 2008 at 12:10 UTC
Updated:
21 Aug 2018 at 03:31 UTC
Jump to comment: Most recent
Comments
Comment #1
Anonymous (not verified) commentedI should have added that I envision the backend cache happening at the time node save. Yes, if this feature were enabled it would slow the save but my data is completely automated with very little user input.
Comment #2
Bèr Kessels commentedAs you admit: this will only help a few. Is the extra code and complexity worth that? Can we not look into a better caching, with intermediate cache systems and/or cron-based re-calulation?
Comment #3
Anonymous (not verified) commentedI can say that my use of tagadelic slowed my site enough that I had to disable the cloud blocks. As it exists today tagadelic isn't functioning well for large sets of tags. Caching at the time of DB change is the only way I know to have real time results. Caching from cron will only slow the cron run and cause other issues since I could add thousands (or more) of rows at once.
Comment #4
Bèr Kessels commentedThis makes sense.
Though looking at other parts of Core Drupal learns me that the "preferred way" is to do it on cron. See search indexing, filter-updates etc.
I prefer to go for (in that particular order)
* the simplest solution
* the solution that behaves most DRupalish
* the solution with least "options"
* the solution that benefits most of us
What do you think, would your case be solved with a cron-based one? Or are you only helped with an onChange solution?
Comment #5
Anonymous (not verified) commentedI don't know. It would depend on how long the process took. I'm thinking that it would help
be helped. If you do add a cron then you need a variable for how many times in a 24 hour period to run it. I execute cron.php once every 5 minutes but wouldn't want to execute this that often.Comment #6
Bèr Kessels commentedFor certain the cron interval zould be configurable. Most (if not all) cron-based-module have such a setting. Default to core cache refresh interval?
Comment #7
Anonymous (not verified) commentedYes, that works.
Comment #8
Bèr Kessels commentedAFAIK the current caching will suffice. Closing issue.
Comment #10
jcisio commentedI reopen the issue because caching is not available in the last stable release (yet, two years ago). Will 6.x-1.3 be out soon?
Cache is suitable for large websites. For those, there are too many nodes and tagadelic data won't change rapidly. I don't know how ofter cache is cleared with CACHE_TEMPORARY, but it shouldn't be cleared often.
Comment #11
Bèr Kessels commentedplease do not reopen. It really was fixed. Just no release yet.