Skip to content

feat: optimize log processing and configuration delivery logic#637

Merged
wodiwudi merged 21 commits intoapache:masterfrom
wtt40122:feat-send-config-async
Mar 5, 2026
Merged

feat: optimize log processing and configuration delivery logic#637
wodiwudi merged 21 commits intoapache:masterfrom
wtt40122:feat-send-config-async

Conversation

@wtt40122
Copy link
Contributor

Ⅰ. Describe what this PR did

  • Reset the logFile stop flag in ChannelServiceImpl to prevent reading from being aborted
  • Added a scheduled cleanup task to DefaultLogProcessCollector to regularly clean up expired data and prevent unlimited memory growth
  • Deduplicate and limit the quantity of collected log processing data to control memory usage
  • Optimize the logic for obtaining log processing status, limit the number of returns, and improve response efficiency
  • Modify the configuration of DefaultPublishConfigService to be asynchronous and support a retry mechanism, improving the stability of sending
  • The newly initialized thread pool replaces the virtual thread executor, enhancing the ability to manage and schedule task distribution
  • Introduce an exponential backoff retry strategy for sending configurations to prevent resource wastage caused by frequent failures
  • Correct initialization parameters and datasets during testing to ensure test coverage and accuracy
  • The version numbers of two projects have been upgraded, indicating new features and improvements

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Copy link
Contributor

@sadadw1 sadadw1 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
Contributor

@wodiwudi wodiwudi left a comment

Choose a reason for hiding this comment

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

LGTM

@wodiwudi wodiwudi merged commit ddc1cee into apache:master Mar 5, 2026
2 of 4 checks passed
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.

3 participants