Skip to content

refactor(analytics): standardize logging and prevent resource leaks#290

Merged
victoriacheng15 merged 1 commit intomainfrom
refactor/analytics-observability
Mar 11, 2026
Merged

refactor(analytics): standardize logging and prevent resource leaks#290
victoriacheng15 merged 1 commit intomainfrom
refactor/analytics-observability

Conversation

@victoriacheng15
Copy link
Owner

Summary

Enhances the analytics engine by enforcing standardized observability practices and ensuring proper resource lifecycle management. These adjustments eliminate unformatted standard output and prevent potential connection leaks during service shutdown, aligning with our operational excellence standards.

List of Changes

  • Standardized initialization error logging to utilize the core telemetry framework, ensuring all startup logs are properly formatted and captured.
  • Implemented explicit connection closure for the secret provider to guarantee a graceful shutdown and prevent resource exhaustion at scale.

Verification

  • Run unit tests for the analytics service to ensure initialization logic is sound.
  • Deploy the service locally and manually verify that graceful shutdown correctly closes the secret store without connection leaks.

@victoriacheng15 victoriacheng15 merged commit d44209d into main Mar 11, 2026
3 checks passed
@victoriacheng15 victoriacheng15 deleted the refactor/analytics-observability branch March 11, 2026 20:27
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.

1 participant