@peterhashair opened this Pull Request on December 20th 2021 Contributor

Description:

Fixes: #17012.
It seems like one of our queries is wrong, update it should work now. But I guess it won't change the previously achieved data.

Review

@peterhashair commented on December 20th 2021 Contributor

It seems like those 2 queries produce 2 different results, I think the second one is correct, which will load the correct average load time, I guess if column is null, it reset sum back to 0?

Current Query


select sum(time_dom_completion+time_dom_processing+time_network+time_on_load+time_server+time_transfer) as total from log_link_visit_action where server_time like '2021-11-08%';

Convert to Query


select sum(+
IFNULL(`time_dom_completion`, 0) 
+IFNULL(`time_dom_processing`,0)+IFNULL(`time_network`,0)+IFNULL(`time_on_load`,0)+IFNULL(`time_server`,0)+IFNULL(`time_transfer`,0))
as total from log_link_visit_action where server_time like '2021-11-08%';
@github-actions[bot] commented on January 6th 2022 Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

This Pull Request was closed on January 7th 2022
Powered by GitHub Issue Mirror