The placement of the Thread.sleep within the try-catch block instead of in a finally block of the APIMonitor run() method leads to a retry loop whenever any exception is thrown - resulting in an explosion of calls from the affected agent to the controller, spiking CPU and memory for both. This fixes this bug by moving the sleep to where it needs to be.
Please make sure these check boxes are checked before submitting
[ ] Squashed Commits
[ ] All Tests Passed - mvn clean test -P default
PR review process
Requires one +1 from a reviewer
Repository owners will merge your PR once it is approved.
The placement of the
Thread.sleep
within the try-catch block instead of in afinally
block of the APIMonitorrun()
method leads to a retry loop whenever any exception is thrown - resulting in an explosion of calls from the affected agent to the controller, spiking CPU and memory for both. This fixes this bug by moving the sleep to where it needs to be.Please make sure these check boxes are checked before submitting
mvn clean test -P default
PR review process