Closed youngzil closed 2 weeks ago
The changes introduced in this pull request encompass enhancements to the Apollo Configuration Center, including the addition of a value length limit function for AppId-level configuration items. Updates include improved server configuration links, REST template client connection pooling, and global search capabilities for administrators. Several fixes address issues with duplicate comments and blank lines. The modifications also enhance the BizConfig
and ItemService
classes to better manage namespace and AppId configurations, alongside updates to documentation for clearer deployment instructions.
File | Change Summary |
---|---|
CHANGES.md |
Added entry for new feature on value length limit for AppId-level configurations, updated server configuration links, and documented several fixes and enhancements. |
apollo-biz/src/main/java/com/ctrip/framework/apollo/biz/config/BizConfig.java |
Added methods for AppId and namespace value length limit overrides, removed an existing method for namespace limits, and introduced a static type reference for deserialization. |
apollo-biz/src/main/java/com/ctrip/framework/apollo/biz/service/ItemService.java |
Updated methods to use Namespace objects instead of long IDs for item value length checks, enhancing clarity and efficiency. |
apollo-biz/src/test/java/com/ctrip/framework/apollo/biz/config/BizConfigTest.java |
Added new test for appIdValueLengthLimitOverride method, including various input scenarios and error handling. |
apollo-biz/src/test/java/com/ctrip/framework/apollo/biz/service/ItemServiceTest.java |
Enhanced test setup with additional dependencies, modified existing tests, and added new tests for item saving with value length limit overrides. |
docs/en/deployment/distributed-deployment-guide.md |
Updated deployment guide with detailed configuration instructions, including MySQL version requirements and Docker/Kubernetes deployment examples. |
docs/zh/deployment/distributed-deployment-guide.md |
Similar updates to the Chinese version of the deployment guide, enhancing clarity and detail on deployment processes and configurations. |
Objective | Addressed | Explanation |
---|---|---|
Provide AppId-level parameter to skip value length checks (#5254) | ✅ | |
Enhance configuration management for value length limits (#5254) | ✅ |
RefreshAdminServerAddressTask
to support dynamic configuration of the time interval, which is a feature highlighted in the main PR.ClusterName
and NamespaceName
fields, which is mentioned as a significant change in the main PR summary.lgtm
🐇 In the garden where configs grow,
Length limits now set to flow.
AppId whispers, "Let me be free,"
With namespaces dancing in harmony.
Updates and fixes, all in a row,
Apollo's magic, watch it glow! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
What's the purpose of this PR
Which issue(s) this PR fixes:
Fixes #5254
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
Release Notes for Apollo Version 2.4.0
New Features
Bug Fixes
Documentation