This will save up to half a minute depending on the system.
Depends on #8532 (comment)
[qa] Adjust poll interval for micro-optimization of run time #8536
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1608-qaOptSync changing 1 files +2 −2-
MarcoFalke commented at 7:32 AM on August 18, 2016: member
-
[qa] Adjust timeouts for micro-optimization of run time fae9567dfc
- MarcoFalke added the label Tests on Aug 18, 2016
-
luke-jr commented at 8:41 PM on August 27, 2016: member
utACK, seems reasonably safe. (The actual timeouts are unchanged, just the poll interval.)
-
MarcoFalke commented at 9:51 PM on August 27, 2016: member
Correct. Actually, increasing the polling frequency increases the timeouts because you have to add the round trip time in each loop to get the "real" timeout.
- MarcoFalke renamed this:
[WIP] [qa] Adjust timeouts for micro-optimization of run time
[WIP] [qa] Adjust poll interval for micro-optimization of run time
on Aug 27, 2016 - MarcoFalke renamed this:
[WIP] [qa] Adjust poll interval for micro-optimization of run time
[qa] Adjust poll interval for micro-optimization of run time
on Sep 9, 2016 -
laanwj commented at 12:33 PM on September 21, 2016: member
Doesn't seem to result in any noticable speedup for me. With this patch:
ALL | True | 832 s (accumulated) Runtime: 243 sWithout this patch:
ALL | True | 838 s (accumulated) Runtime: 237 s - MarcoFalke closed this on Sep 21, 2016
-
MarcoFalke commented at 12:50 PM on September 21, 2016: member
Closing, I think there are better ways to improve runtime.
- MarcoFalke deleted the branch on Sep 21, 2016
- MarcoFalke locked this on Sep 8, 2021
Contributors
Labels