Error in pydoc for storm.store.Store related to default cache size
Bug #374180 reported by
Drew Smathers
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Storm |
Fix Released
|
Medium
|
Jamu Kakar | ||
Bug Description
Documentation for Store initializer states:
@param cache: The cache to use. Defaults to a L{Cache} storing up to
1000 objects at any given time.
However, Cache is initialized with 100 by default.
Revision history for this message
| Drew Smathers (djfroofy) wrote | #1 |
Revision history for this message
| James Henstridge (jamesh) wrote | #2 |
Revision history for this message
| James Henstridge (jamesh) wrote | #3 |
| Changed in storm: | |
| milestone: | none → 0.16 |
| Changed in storm: | |
| importance: | Undecided → Medium |
| assignee: | nobody → Jamu Kakar (jkakar) |
Revision history for this message
| Jamu Kakar (jkakar) wrote | #4 |
| Changed in storm: | |
| assignee: | Jamu Kakar (jkakar) → nobody |
| status: | New → Fix Committed |
| Changed in storm: | |
| assignee: | nobody → Jamu Kakar (jkakar) |
| Changed in storm: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Looking at the last revision to change that line of the documentation, it looks like the intent might have been to change the default cache size:
http://
Gustavo: would the correct fix here be to change Store.__init__() to not set a size on the cache, leaving the default to the Cache constructor?