[gui][server] Statistics #2897
[gui][server] Statistics #2897
Conversation
csordasmarton
commented
Aug 19, 2020
csordasmarton
commented
- Show statistics in separate tabs.
- Component statistics.
- Compare statistics.
- Improve compare queries by using subqueries.
csordasmarton
added
enhancement 🌟
GUI 🎨
server 🖥️
refactoring 😡 ➡️ 🙂
| @@ -349,6 +347,81 @@ def process_report_filter(session, report_filter): | |||
| return filter_expr | |||
|
|
|||
|
|
|||
| def get_outstanding_reports_filter(tbl=Report): | |||
There was a problem hiding this comment.
this was renamed to open reports right?
what is the tbl=Report expression, Report is a type right?
| or_(report_cnt_q.c.fixed_at.is_(None), | ||
| report_cnt_q.c.fixed_at >= | ||
| RunHistory.time))) \ | ||
| .filter(get_outstanding_reports_filter(report_cnt_q.c)) \ |
There was a problem hiding this comment.
open reports filter? Shouldn't the query from the other PR be used when that is merged?
csordasmarton
force-pushed
the
refactore_statistics
branch
from
00edcb3 to
afd8115
Compare
- Show the number of reports based on the compare to (newcheck) section when diffing statistics and show the number of resolved and new reports beside it. Show these numbers as links so the user can jump and see the reports. - Hide diff type on statistics view.
If compare data was set on the server side the queries were really slow because we queried the bug hashes first, then we used these hashes in an IN clause. This patch will use subqueries to filter the results. It will drastically improve the performance.
csordasmarton
force-pushed
the
refactore_statistics
branch
from
afd8115 to
8538728
Compare
|
Issues
======
+ Solved 1
- Added 41
Complexity increasing per file
==============================
- web/server/vue-cli/src/components/Statistics/StatisticsHelper.js 3
Complexity decreasing per file
==============================
+ web/server/codechecker_server/api/report_server.py -2
Clones removed
==============
+ web/server/codechecker_server/api/report_server.py -11
See the complete overview on Codacy |
There was a problem hiding this comment.
I've tested some use cases and found some interesting behavior.
click on the statistics/component statistics tab and there is let's say 2 confirmed bugs for a component.
click on the number 2
the UI jumps to the reports tab but the reports are not filtered based on the checker name the review status and the source component, I see much more results not just the 2 which were expected.
Is this behavior known and will be resolved in #2913?
