If a view no longer contains that node; this would allow for that to be updated.

CommentFileSizeAuthor
#4 boost-601086.patch8.28 KBmikeytown2

Comments

mikeytown2’s picture

Store the time stamp too; will allow for smarter operations if the view shrinks in size. If the time stamp for that entry doesn't match the rest, then its old and needs to be cleared. Something like 5x older then the expiration. Older then 3 days would be the minimum. Trim on cron; update on cache creation.

mikeytown2’s picture

Better yet, instead of flushing it, check the actual view first; only if the view no longer contains the node will that entry be removed.

mikeytown2’s picture

Best idea today... remove the entires from the relationship table when that view is flushed. Still need the other parts of this though, just makes the odds of something falling through the cracks even less.

mikeytown2’s picture

Status: Active » Needs review
StatusFileSize
new8.28 KB

Covers all cases expect for when a view shrinks in size (using pager; url no longer valid)... will have to cover that somehow; most likely a timestamp db scan on cron. Open to idea on this one. Alt would be a hook for views inside boost_cache_kill(), but thats not 100% accurate then.

mikeytown2’s picture

Status: Needs review » Fixed

committed

Status: Fixed » Closed (fixed)

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