Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(177)

Issue 186208: Don't put many2one in the cache (too slow) and externalize some function result outside the loop (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years ago by ced
Modified:
16 years ago
Reviewers:
bch, h.goebel, yangoon1
Visibility:
Public.

Patch Set 1 #

Total comments: 4

Patch Set 2 : Remove unused code and better iterator naming #

Patch Set 3 : Better line break #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -27 lines) Patch
M trytond/model/modelsql.py View 1 2 1 chunk +14 lines, -27 lines 0 comments Download

Messages

Total messages: 5
ced
16 years ago (2010-01-18 19:57:55 UTC) #1
h.goebel
http://codereview.appspot.com/186208/diff/1/2 File trytond/model/modelsql.py (right): http://codereview.appspot.com/186208/diff/1/2#newcode1221 trytond/model/modelsql.py:1221: .setdefault(self._name, set()) putting the ".setdefault" into the next line ...
16 years ago (2010-01-19 10:04:15 UTC) #2
ced
16 years ago (2010-01-19 10:12:02 UTC) #3
yangoon1
16 years ago (2010-01-19 10:25:57 UTC) #4
ced
16 years ago (2010-01-20 14:50:11 UTC) #5

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b