Summary of Changes

Hello @dhth, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request focuses on optimizing the performance and responsiveness of the forwarder's worker processes by removing unnecessary sleep intervals. By eliminating the explicit pauses in both the upload and report workers, the system can now react more quickly to incoming tasks and process data continuously without artificial delays. This change also includes a cleanup of related configuration parameters and improves the clarity of startup logging.

Highlights

• Elimination of Worker Sleep Configuration: The environment variable KPLAY_FORWARD_UPLOAD_WORKER_SLEEP_MILLIS and its associated default, minimum, and maximum values have been entirely removed from the configuration. This simplifies the forwarder's setup by removing an unnecessary tuning parameter.
• Removal of Worker Sleep Logic: Explicit time.Sleep calls within both the upload worker (startUploadWorker) and reporter worker (startReporterWorker) loops have been removed. This change allows these workers to continuously process tasks without artificial delays, potentially improving responsiveness and throughput.
• Refactored Logging for Clarity: The logging for upload_reports and run_server has been updated to provide more explicit information. It now clearly indicates true or false for these behaviors and includes related parameters (like report_batch_size or host/port) on the same log line for better readability.