ZStorm provides no mechanism to change the default Cache
Bug #391996 reported by
Stuart Bishop
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
Low
|
Unassigned | ||
Storm |
Triaged
|
Medium
|
Unassigned |
Bug Description
Neither ZStorm.get nor ZStorm.create allow you to override the Cache being used.
I think both of these methods should be extended to allow an optional Store factory to be passed in, defaulting to storm.store.Store. This allows people who need to to override the default Cache implementation and size, as well as handling any future options the Store might grow in the future.
Related branches
lp://staging/~stub/storm/bug-391996
Rejected
for merging
into
lp://staging/storm
- Jamu Kakar (community): Abstain
- James Henstridge: Needs Fixing
- Diff: 141 lines (has conflicts)
Changed in storm: | |
milestone: | 0.15 → none |
Changed in storm: | |
milestone: | none → 0.16 |
tags: | added: tech-debt |
Changed in storm: | |
milestone: | 0.16 → 0.17 |
Changed in storm: | |
milestone: | 0.17 → 0.18 |
Changed in storm: | |
milestone: | 0.18 → 0.19 |
Changed in storm: | |
assignee: | Stuart Bishop (stub) → nobody |
status: | In Progress → Triaged |
milestone: | 0.19 → none |
To post a comment you must log in.
Merge proposal is up for review.