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.

Sign up for GitHub

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

Jump to bottom

[Update] add bypass_values for values aren't wanted to cache in redis. #4

Closed
wants to merge 2 commits into from
Closed

[Update] add bypass_values for values aren't wanted to cache in redis. #4

wants to merge 2 commits into from

Conversation

fatelei
Copy link
Contributor

@fatelei fatelei commented May 2, 2016 

self._raw_get return `null`

@fatelei fatelei force-pushed the fix/get_none branch 2 times, most recently from 271a7fc to f5dde73 Compare May 2, 2016 02:29
@fengsp
Copy link
Owner

fengsp commented May 2, 2016

I have an idea, we provide one parameter called bypass_values for the cache object, which is a list of values that won't be cached at all.

@fatelei fatelei changed the title [Fix] self._raw_get return 'null' [Update] add bypass_values for values aren't wanted to cache in redis. May 2, 2016
@fatelei
Copy link
Contributor Author

fatelei commented May 2, 2016

image

All tests passed

@fengsp
Copy link
Owner

fengsp commented May 3, 2016

There is one bug there, and the test example is kind of meaningless, and no docs. Let me do it.

@fengsp
Copy link
Owner

fengsp commented May 3, 2016

http://rc.readthedocs.io/en/latest/cache.html#bypass-values

@fengsp fengsp closed this May 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fatelei @fengsp