Closed Rawven closed 1 month ago
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅
I have read the CLA Document and I hereby sign the CLA
## Walkthrough
The changes in this pull request involve an update to the `CHANGES.md` file, specifically the addition of a new entry for Apollo Java version 2.4.0. This entry documents the feature "Add more observability in apollo config client," which is linked to pull request #74. The update is part of the release notes and is positioned alongside existing entries that cover various fixes and features related to configuration bean definitions and open API query namespace support.
## Changes
| Files | Change Summary |
|------------------|---------------------------------------------------------------------------------------------------------|
| `.../CHANGES.md` | Added new entry: "Add more observability in apollo config client" for Apollo Java version 2.4.0. |
## Possibly related PRs
- #78: This PR updates the version to 2.4.0-SNAPSHOT and reflects changes in the `CHANGES.md` file, which is directly related to the addition of the new entry for "Add more observability in apollo config client" in the main PR.
> 🐇 "In the garden, changes sprout,
> Observability, there's no doubt!
> Apollo's notes now sing with glee,
> Config insights, wild and free!
> With every hop, we celebrate,
> New features bloom, oh, isn’t it great!" 🌼
@Anilople PTAL
Congratulation to submit the pr, that will be a great start.
A check failed, need to fix it
Attention: Patch coverage is 64.12121%
with 296 lines
in your changes missing coverage. Please review.
Project coverage is 68.07%. Comparing base (
1fc9d27
) to head (60958c4
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The overall design becomes complete and unit tests begin to be written.
@apolloconfig/committers hello, guys, after a long time review and code change, I think it time to merge this pr.
Need help, please take a look, is there something to watch out or change, or other suggestion?
@nobodyiam Sir, the unnecessary modifications and formatting issues have been resolved.
What's the purpose of this PR
discussions OSPP2024 test cases Test
Which issue(s) this PR fixes:
Issue
Brief changelog
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
mvn clean test
to make sure this pull request doesn't break anything.CHANGES
log.Summary by CodeRabbit
Summary by CodeRabbit
New Features
ApolloClientBootstrapArgsMonitorApi
,ApolloClientNamespaceMonitorApi
, andApolloClientThreadPoolMonitorApi
to manage configuration parameters and resource metrics.ConfigMonitorInitializer
class to facilitate the setup of monitoring components.Improvements
Bug Fixes
Tests
ApolloClientMonitorContext
,ApolloClientMetricsExporter
, and other monitoring components to ensure correct functionality.