The new Ming build does stricter type checking, causing some Allura tests to fail. Need to update Allura code to fix the failures before we can move to the new Ming.
If you want to run the tests or browse allura, you'll need to install the new version of Ming (http://downloads.sourceforge.net/project/merciless/0.2.2dev-20120210/Ming-0.2.2dev-20120210.tar.gz) in your allura virtualenv.
In my testing of the Allura Tracker home page with a dataset from prod, page load time went from ~4.6s on dev to ~2.4s on this branch.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
forge:tv/3830
Mainly just a code review - all the tests pass.
If you want to run the tests or browse allura, you'll need to install the new version of Ming (http://downloads.sourceforge.net/project/merciless/0.2.2dev-20120210/Ming-0.2.2dev-20120210.tar.gz) in your allura virtualenv.
In my testing of the Allura Tracker home page with a dataset from prod, page load time went from ~4.6s on dev to ~2.4s on this branch.
To ssh://engr.geek.net/forge
+ 7c2ded0...09db174 tv/3830 -> tv/3830 (forced update)
Updated Ming to truncate DateTimes at milliseconds instead of seconds. New build is here: http://downloads.sourceforge.net/project/merciless/0.2.2dev-20120302/Ming-0.2.2dev-20120302.tar.gz
Updated tv/3820 to remove the now-unnecessary
sleep(1)from the mod_date test.To ssh://engr.geek.net/forge
09db174..6e955d6 tv/3830 -> tv/3830
Updated Ming to truncate microseconds from DateTimes but preserve milliseconds. New build is here: http://downloads.sourceforge.net/project/merciless/0.2.2dev-20120305/Ming-0.2.2dev-20120305.tar.gz
Updated tv/3830 to use new Ming build. All Allura tests pass now.
Looks good, new version on internal repo, tests pass. Merged.