Merge lp://qastaging/~divmod-dev/divmod.org/1209082-remove-bogus-performance-tests into lp://qastaging/divmod.org
Proposed by
Tristan Seligmann
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 2715 | ||||
| Proposed branch: | lp://qastaging/~divmod-dev/divmod.org/1209082-remove-bogus-performance-tests | ||||
| Merge into: | lp://qastaging/divmod.org | ||||
| Diff against target: |
159 lines (+10/-79) 1 file modified
Axiom/axiom/test/test_query.py (+10/-79) |
||||
| To merge this branch: | bzr merge lp://qastaging/~divmod-dev/divmod.org/1209082-remove-bogus-performance-tests | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Laurens Van Houtven | Approve | ||
|
Review via email:
|
|||
Commit message
Remove performance tests that relied on the ordering of tables to influence the query planner.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I think that the discussion resulted in the conclusion that these particular tests are untenable, and removing them is the appropriate way forward. (Specifically, we can't make them pass on all SQLites we support, because Py2.6 and Py2.7 sqlite3 modules have different behavior...). Please merge.