Skip to content

feat(service-discovery): 稳定应用级服务发现使用的元数据key#3362

Open
Alanxtl wants to merge 2 commits into
apache:developfrom
Alanxtl:stablize_register_services
Open

feat(service-discovery): 稳定应用级服务发现使用的元数据key#3362
Alanxtl wants to merge 2 commits into
apache:developfrom
Alanxtl:stablize_register_services

Conversation

@Alanxtl
Copy link
Copy Markdown
Contributor

@Alanxtl Alanxtl commented Jun 2, 2026

背景

应用级服务发现集合 listener key 之前依赖 services.String(),相同应用集合在不同遍历顺序下可能生成不同 key,导致 listener 复用不稳定。

改动内容

  • services.String() 替换为排序后的稳定 application set key。

价值

  • 提升应用级服务发现 listener key 的稳定性,避免相同应用集合产生不同订阅 key。

Checklist

  • I confirm the target branch is develop
  • Code has passed local testing
  • I have added tests that prove my fix is effective or that my feature works

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

@Alanxtl Alanxtl changed the title feat(service-discovery): 稳定服务注销后的应用级元数据 feat(service-discovery): 稳定应用级服务发现使用的元数据key Jun 2, 2026
@Alanxtl Alanxtl added ✏️ Feature 3.3.2 version 3.3.2 labels Jun 2, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.53%. Comparing base (60d1c2a) to head (0aa91ec).
⚠️ Report is 814 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3362      +/-   ##
===========================================
+ Coverage    46.76%   52.53%   +5.76%     
===========================================
  Files          295      492     +197     
  Lines        17172    37878   +20706     
===========================================
+ Hits          8031    19898   +11867     
- Misses        8287    16374    +8087     
- Partials       854     1606     +752     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants