[Kde-pim] Review Request 121307: Add a list of queries to the top of the query debugger.

[Daniel Vrátil]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121307/#review71566
-----------------------------------------------------------


I've actually used AC with this patch. Works fine, but I found a bug, and couple places for improvements. Please fix the bug, and consider implementing the features below. Otherwise just ship the patch, and I'll push my changes :)

Bug:
- "Clear View" context menu for the query view (the old one) is mispositioned now

Improvements:
- setDynamicSortFilter(true), otherwise you have to re-sort manually all the time (I'm aware this means much more CPU usage, but hey, it's Akonadi Console, nobody cares :)
- add a column with average query duration (QueryInfo::duration / QueryInfo::calls) - makes it easier to find actually slow queries
- a fake "TOTAL" row with total sum of duration and calls and average of average durations - just for information (also makes it possible to actually count some percentages or see the "big picture")
- action to clear the model - to collect statistics about a particular event, you probably want to start from 0, but restarting Akonadi Console all the time just to clear the previous queries is annoying :)
- use a QSplitter to make the views resizable (usability in Akonadi Console, whooaaa!!)

- Daniel Vrátil


On Dec. 8, 2014, 10:34 a.m., Milian Wolff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121307/
> -----------------------------------------------------------
> 
> (Updated Dec. 8, 2014, 10:34 a.m.)
> 
> 
> Review request for KDEPIM, Daniel Vrátil and Volker Krause.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> This list aggregates the query runtime and number of calls to the
> query. This is much more useful, imo, than the text view below.
> 
> Note that this patch also disables the text view by default, as it
> can easily lead to out-of-memory issues. Blindly appending lots
> of text is bound to fail, sooner or later.
> 
> REVIEW: 121307
> 
> Add reviewboardrc file.
> 
> 
> Diffs
> -----
> 
>   .reviewboardrc PRE-CREATION 
>   akonadiconsole/querydebugger.h 52b8a2d57501cd742dbc18516f83fe74bc7b19e0 
>   akonadiconsole/querydebugger.cpp ac1d118a0a8eebb56521f50b5fb1e9b03dec47ff 
> 
> Diff: https://git.reviewboard.kde.org/r/121307/diff/
> 
> 
> Testing
> -------
> 
> it works!
> 
> 
> File Attachments
> ----------------
> 
> screenshot of aggregation
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/12/01/a3b406a4-d03d-41dc-b198-4f2f304208cb__akonadiconsole_debugger.png
> 
> 
> Thanks,
> 
> Milian Wolff
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/