[YUNIKORN-2409] History tracking misc code cleanup

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Labels: pull-request-available
• Target Version: 1.5.0
• Assignee: Peter Bacsko
• Reporter: Peter Bacsko
• Created: 12/Feb/24 13:20
• Updated: 20/Mar/24 14:32
• Resolved: 12/Feb/24 22:37

Description

Various parts of the app history tracking can be improved/cleaned up:

1. event_streaming.go: removed unused variable "count"
2. ring buffer: unnecessary rlock/runlock in unexported method 
   getEventsFromID()
3. REST handler getStream() function: use deferred call to eventSystem.RemoveStream()
4. application_events.go: limiter is no longer used

Issue Links

• links to: GitHub Pull Request #797