Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aspect Oriented Programming style Caching with Castle Windsor #1

Open
LodewijkSioen opened this issue Dec 11, 2013 · 7 comments
Open

Aspect Oriented Programming style Caching with Castle Windsor #1

LodewijkSioen opened this issue Dec 11, 2013 · 7 comments
Labels

Comments

@LodewijkSioen
Copy link
Owner

@LodewijkSioen LodewijkSioen commented Dec 11, 2013

No description provided.

@petermorlion
Copy link

@petermorlion petermorlion commented Dec 12, 2013

Very cool! Are you thinking about handling cache invalidation or is it not necessary in your case? Would make for a great follow up post (hint hint ;)

@LodewijkSioen
Copy link
Owner Author

@LodewijkSioen LodewijkSioen commented Dec 12, 2013

Well, it's all academic. We're not actually using this :)

It is possible to give a cache-timeout in the Enterprise Edition on github:

 [Cached(TimeoutInMinutes = 5, CacheKeyGenerator = typeof(DummyCacheKeyGenerator), CacheRegion = "region")]
public virtual DateTime CachedMethodWithSettings(string argumentOne, int argumentTwo)
{    
    return DateTime.Now;
}
@petermorlion
Copy link

@petermorlion petermorlion commented Dec 12, 2013

Am I correct that the DefaultCacheProvider doesn't check this at the moment?

@LodewijkSioen
Copy link
Owner Author

@LodewijkSioen LodewijkSioen commented Dec 12, 2013

On line 41 an item is added to the cache with an absolute expiration. The cache implementation itself is responsible for invalidation.

@petermorlion
Copy link

@petermorlion petermorlion commented Dec 12, 2013

Yes, I'd seen that. Just wasn't sure if I was missing anything. So I understood it correctly. Cool, thanks! Nice, clean implementation by the way!

@LodewijkSioen
Copy link
Owner Author

@LodewijkSioen LodewijkSioen commented Dec 12, 2013

Thanks! Haven't actually used it though, so it's all just an exercise now.

@petermorlion
Copy link

@petermorlion petermorlion commented Dec 12, 2013

Exercise is good :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants