benchmark file added + add_results and update_results performance imporvement

MolSSI / QCFractal

A distributed compute and database platform for quantum chemistry.

https://molssi.github.io/QCFractal/

BSD 3-Clause "New" or "Revised" License

148 stars 48 forks source link

benchmark file added + add_results and update_results performance imporvement #573

Closed codtiger closed 4 years ago

codtiger commented 4 years ago

Description

Changed the add_results and update_results function in sqlalchemy_socket.py to do only one commit in case of non-repeated inserts. Benchmark file also provided for comparison. The performance comparisons are also attached. runs.txt

Changelog description

add_results , update_results slight performance improvement

Status

[ ] Code base linted
[ ] Ready to go

lgtm-com[bot] commented 4 years ago

This pull request introduces 3 alerts when merging 69ca919b1ca1c7320b39646afde471667522c5d6 into 137cb91d4409a1395273239a9df668a314a1914b - view on LGTM.com

new alerts:

2 for Unused local variable
1 for Unreachable code