Error in pydoc for storm.store.Store related to default cache size

Bug #374180 reported by Drew Smathers on 2009-05-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Storm
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.

Drew Smathers (djfroofy) wrote :
James Henstridge (jamesh) wrote :

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?

James Henstridge (jamesh) wrote :

Definitely should address this for 0.16.

Changed in storm:
milestone: none → 0.16
Jamu Kakar (jkakar) on 2009-11-29
Changed in storm:
importance: Undecided → Medium
assignee: nobody → Jamu Kakar (jkakar)
Jamu Kakar (jkakar) wrote :

Merged to storm/trunk r347 [trivial].

Changed in storm:
assignee: Jamu Kakar (jkakar) → nobody
status: New → Fix Committed
Jamu Kakar (jkakar) on 2009-11-29
Changed in storm:
assignee: nobody → Jamu Kakar (jkakar)
Jamu Kakar (jkakar) on 2009-11-29
Changed in storm:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers