search

tnc-ba / strongTNC

BYOD TNC Database Management Tool
GNU Affero General Public License v3.0
0 stars 0 forks source link

Performance Issues #294

Closed dbrgn closed 10 years ago

dbrgn commented 10 years ago
dbrgn commented 10 years ago

Work in progress: https://github.com/tnc-ba/strongTNC/tree/performance_issues

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.21%) when pulling a2ebc2a2099a5ba73df2852ad673093f8e74e1b7 on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.21%) when pulling 1687b46bbd83f3aec45103b061bb73cac9de5b5c on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

d22 commented 10 years ago

Found one issue (see file/line comments). Will do more testing tomorrow....

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.25%) when pulling a76ddc2414bc76a9afbf29b394b6a5b2e85cc30f on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.25%) when pulling c40750ec05b967c4d66b84271344e979685bc9dc on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.25%) when pulling 1caa8f3be51bbb69678e522fa7f52ba98c68ecae on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.22%) when pulling dfc43b72cad5952907a409eb9228e7d9743d8517 on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.26%) when pulling aadfab1b16efba0d7901cd7efe6b18008bd4214b on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

cfaessler commented 10 years ago

@d22 and @dbrgn what do you think, how shall we proceed?

d22 commented 10 years ago
coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.31%) when pulling fdefb77d3651c20e63589674c3b17ae0cb8748ab on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

d22 commented 10 years ago

I fixed the "to many SQL variables" problem in the log view and added a filter to the log view. Pls, test and review.

I'm currently working on the "inital measurement is not shown in log view" problem. I hope I can finish this tomorrow.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.74%) when pulling a53c5f37fab5374d7a41bc5897433e51aa8eb6c3 on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

d22 commented 10 years ago

Tnx for the defer('swid_xml') hint, totally forgot about it.

The inital measurement is now shown in the log view as well. If the last session in the timerange is not the initial measurement, all additions from this sessions are shown as well.

Please review and test. Tnx!

d22 commented 10 years ago

Btw. I also tested the the API and views with "BIG DATA" and found no further bugs so far. All I found was a gigantic performance boost! :godmode: :star2: :heart:

dbrgn commented 10 years ago

Migration instructions:

1. Create new database table

sqlite3 /path/to/database
sqlite> .read sql/issue294_create.sql

2. Migrate existing sessions

./manage.py shell
>>> execfile('sql/issue294_migrate.py')
coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.74%) when pulling c8282a6296a0027240c8b4f8ef9b9f4c7f636404 on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.74%) when pulling 525adc2ddcde94307b8d37906313a72fda1f44f6 on performance_issues into 133cb24d96073415f308a10f947199c3ce15cec6 on master.