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.
Changed in storm: | |
importance: | Undecided → Medium |
assignee: | nobody → Jamu Kakar (jkakar) |
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:// bazaar. launchpad. net/~storm/ storm/trunk/ revision/ 292
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?