Code review comment for lp:~vauxoo/openobject-addons/trunk-bug_887376-account_move_compute_with_btree-moylop260
Revision history for this message
| Nhomar - Vauxoo (nhomar) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
Hello Oliver.
I'm sorry, i didn't understand before your point about WORK IN PROGRESS!
The intention of this merge in this way IS NOT a proof of concept, we have using it for 6 month in production.
All TODO's are the original ones in the code we JUST optimize the core SQL code.
IOH, the reason because we left both methods is to help to testers to use BOTH methods compare time and over all RESULTS.
Don't you have a real db with real data to tests purpose on your enviroment?, this feature is so so important and we need mae test.
As you can see it is just ONE method for this reason, i don't face problems in for RC1 left both people make tryies and then we take all, do you have my point friend?
Regards!
PS: If you answer me that you can not make merge to RC 1 for this reason i can correct the merge, but i want to make live easyiest to testers.
Regards.