Merge lp:~jkakar/storm/is-empty-strips-order-by into lp:storm
Proposed by
Jamu Kakar
Status: | Merged |
---|---|
Approved by: | Jamu Kakar |
Approved revision: | 362 |
Merged at revision: | 362 |
Proposed branch: | lp:~jkakar/storm/is-empty-strips-order-by |
Merge into: | lp:storm |
Diff against target: |
52 lines (+23/-1) 2 files modified
storm/store.py (+1/-0) tests/store/base.py (+22/-1) |
To merge this branch: | bzr merge lp:~jkakar/storm/is-empty-strips-order-by |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Kevin McDermott (community) | Approve | ||
Robert Collins (community) | Approve | ||
Storm Developers | Pending | ||
Review via email: mp+30677@code.launchpad.net |
Description of the change
This branch introduces the following changes:
- ResultSet.is_empty strips order by clauses, since they're
unnecessary for this operation.
To post a comment you must log in.
Looks good except for the typo ;)