REGRESSION: Perf-o-matic adds duplicated test results
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2012 05:45:08 +0000 (05:45 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2012 05:45:08 +0000 (05:45 +0000)
commitba5d8b944787c3616534ea86e22be5b8370328ae
tree3e549b88670c7dec86a2dd6535928278fdb66e61
parent522292435e9e582f67069d839801fa9ec6cdff7f
REGRESSION: Perf-o-matic adds duplicated test results
https://bugs.webkit.org/show_bug.cgi?id=78373

Reviewed by Hajime Morita.

Add TestResult inside a transaction. We use high replication databsae but this should prevent
the duplication in the most of the time.

Also fixed random bugs in Controller and ReportHandler.

* Websites/webkit-perf.appspot.com/controller.py:
(get_persistent_cache):
* Websites/webkit-perf.appspot.com/report_handler.py:
(ReportHandler.post):
* Websites/webkit-perf.appspot.com/report_process_handler.py:
(ReportProcessHandler.post):
(ReportProcessHandler._add_test_if_needed):
(ReportProcessHandler):
(ReportProcessHandler._add_test_result_if_needed):
(ReportProcessHandler._add_test_result_if_needed._float_or_none):
(ReportProcessHandler._add_test_result_if_needed.execute):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@107531 268f45cc-cd09-0410-ab3c-d52691b4dbfc
ChangeLog
Websites/webkit-perf.appspot.com/controller.py
Websites/webkit-perf.appspot.com/models.py
Websites/webkit-perf.appspot.com/report_handler.py
Websites/webkit-perf.appspot.com/report_process_handler.py