Skip to content

[feat][ETS][monitor] optimize monitor module with log cleanup and connection pool expansion#976

Merged
casionone merged 7 commits intodev-2.0.0from
dev-2.0.0-monitor-update
Apr 7, 2026
Merged

[feat][ETS][monitor] optimize monitor module with log cleanup and connection pool expansion#976
casionone merged 7 commits intodev-2.0.0from
dev-2.0.0-monitor-update

Conversation

@v-kkhuang
Copy link
Copy Markdown

What is the purpose of the change

Background/Problem:
The current Monitor module lacks automatic log cleanup mechanisms, has limited connection pool capacity, and diagnosis functions need refactoring for better maintainability. These limitations affect system performance and operational efficiency.

Purpose of Change:
To improve the Monitor module, this PR adds automatic diagnosis log cleanup, refactors diagnosis functions, and expands the connection pool to handle higher concurrency loads.

Value/Impact:
After the change, Monitor module has better maintainability with automated log cleanup, improved diagnosis capabilities, and enhanced performance through connection pool optimization.

Related issues/PRs

Related issues: close #975
Related pr:none

Brief change log

  • Add automatic diagnosis log data cleanup functionality
  • Optimize monitor parameters for better performance
  • Refactor diagnosis function for improved modularity
  • Expand connection pool capacity for higher concurrency

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

@v-kkhuang v-kkhuang added the enhancement New feature or request label Apr 7, 2026
Copy link
Copy Markdown

@casionone casionone left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Copy Markdown

@casionone casionone left a comment

Choose a reason for hiding this comment

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

LGTM.

@casionone casionone merged commit 297faf4 into dev-2.0.0 Apr 7, 2026
8 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants