[ZBX-11482] httpconf.php uses infinite backlook in getLast() Created: 2016 Nov 14 Updated: 2017 May 30 Resolved: 2017 Jan 04 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | API (A) |
Affects Version/s: | 3.0.5, 3.2.0 |
Fix Version/s: | 3.0.8rc1, 3.2.4rc1, 3.4.0alpha1 |
Type: | Incident report | Priority: | Trivial |
Reporter: | D.Spindel Ljungmark | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | lastvalue, patch, performance, webmonitoring | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: |
![]() |
Description |
httpconf.php uses infinite backlooking on history tables to find a single value, where other parts of the code bounds the lookback to ZBX_HISTORY_PERIOD, usuall 86400 By bounding the query we can avoid an expensive backlook with an unbounded time if the history_uint table is big. |
Comments |
Comment by D.Spindel Ljungmark [ 2016 Nov 14 ] |
Patch that limits how far back in time the query looks. |
Comment by vitalijs.cemeris (Inactive) [ 2016 Nov 28 ] |
(1) Translation strings removed:
gunarspujats CLOSED |
Comment by vitalijs.cemeris (Inactive) [ 2016 Nov 28 ] |
Fixed in the development branch svn://svn.zabbix.com/branches/dev/ZBX-11482 |
Comment by Gunars Pujats (Inactive) [ 2016 Dec 05 ] |
(2) Now if web test is not executed in last 24 hours, then status is "Unknown" and field "Last check" is "Never". vitalijs.cemeris RESOLVED in r64509 gunarspujats CLOSED |
Comment by Gunars Pujats (Inactive) [ 2016 Dec 19 ] |
(3) Coding style RESOLVED in r64570 vitalijs.cemeris CLOSED |
Comment by Gunars Pujats (Inactive) [ 2016 Dec 19 ] |
(4) [D] In Monitoring->Web list and details views fields Last check and Status will be empty if web scenario is not executed in last 24 hours. This should be mentioned in vitalijs.cemeris RESOLVED martins-v Thanks, I also added "since Zabbix 3.0.8/3.2.4" to the corresponding entries. Mentioned this in the upgrade notes for 3.0.8 and 3.2.4 as well. Could be closed now. vitalijs.cemeris CLOSED |
Comment by Gunars Pujats (Inactive) [ 2016 Dec 19 ] |
TESTED, but close (3) before merging |
Comment by vitalijs.cemeris (Inactive) [ 2017 Jan 02 ] |
Fixed in:
|