No clean way to get the underlying Storm ResultSet from an SQLObjectResultSet
Bug #338184 reported by
Gavin Panella
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Storm |
Fix Released
|
Undecided
|
Gavin Panella | ||
Bug Description
In Launchpad, because we use both the native Storm API and have lots of older code using the SQLObject shim, we sometimes need to get at the underlying Storm ResultSet from an SQLObjectResultSet. That's possible via so_result_
For a Zope environment, one proposed solution (from Francis Lacoste) is to create a trusted adapter from ISQLObjectResultSet to IResultSet.
Related branches
lp:~allenap/storm/adapt-sqlobject-to-storm
- Gustavo Niemeyer: Approve
- James Henstridge: Approve
- Diff: 185 lines
Revision history for this message
| James Henstridge (jamesh) wrote | #1 |
| Changed in storm: | |
| assignee: | nobody → allenap |
| status: | New → In Progress |
Revision history for this message
| James Henstridge (jamesh) wrote | #2 |
| Changed in storm: | |
| status: | In Progress → Fix Committed |
| Changed in storm: | |
| milestone: | none → 0.15 |
| Changed in storm: | |
| status: | Fix Committed → Fix Released |
| tags: | added: tech-debt |
To post a comment you must log in.
So I guess this is intended as a way to gradually migrate callers of a heavily used API so that it can then be switched to Storm native?
If it makes it easier for you to get off the compatibility layer, I'm all for it. I've created a merge proposal for your branch so we can review it.