Skip to content

Conversation

@notshivansh
Copy link

No description provided.

Copilot AI review requested due to automatic review settings September 25, 2025 11:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the log monitoring system to improve logging efficiency and state tracking. The changes replace a simple time-based approach with a more sophisticated timestamp tracking mechanism for individual log streams.

  • Introduces a new TimestampTracker for per-stream timestamp management with automatic cleanup
  • Modifies log stream processing to track individual stream positions instead of global timestamps
  • Enhances debug logging throughout with more detailed context information

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
logprocesser/logreader.go Major refactoring to implement per-stream timestamp tracking, improved logging, and better state management
.github/workflows/main.yml Updates Docker tag default value and adds description

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@notshivansh notshivansh merged commit 4bb7f92 into feature/api_gateway_logging_temp_cred Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants