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

Cache settings #4125 #4128

Merged
merged 9 commits into from
Jul 29, 2015
Merged

Conversation

benjaoming
Copy link
Contributor

Builds upon #4122

Fixes one main issue:

  • File based caching is not persistent because it's stored in a tempdir!

New feature:

  • The dev settings now include memory based caching by default
  • Docs mention need to clear webcache at one of the settings where it's necessary
  • Settings for caching much less messy
  • More efficient topic_tree view now that it doesn't have to write the topic tree to a useless file-based cache
    Default cache settings are weird #4125

This was referenced Jul 20, 2015
@benjaoming benjaoming mentioned this pull request Jul 27, 2015
8 tasks
@benjaoming benjaoming added this to the 0.14.0 milestone Jul 27, 2015
@benjaoming
Copy link
Contributor Author

@MCGallaspy assigning you because it's an easy task IMO... just check that the code doesn't have any obvious problems... it's been tested in 3 alphas now without issues.

@rtibbles
Copy link
Member

Seems OK to me, and has been tested.

rtibbles added a commit that referenced this pull request Jul 29, 2015
@rtibbles rtibbles merged commit b8692f5 into learningequality:0.14.x Jul 29, 2015
@rtibbles rtibbles removed the has PR label Jul 29, 2015
@benjaoming benjaoming deleted the cache-settings branch August 19, 2015 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants