optimize: metadata discovery support for redis #7756
+77
−28
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ⅰ. Describe what this PR did
Enhance the Redis registry center with metadata registration and discovery capabilities based on branch 'gsoc-2025-meta-registry'.
1. Storage Design
Survival Key:
registry.redis.{cluster}_{ip:port}(String,SETEXfor keep-alive, heartbeat renewal)Metadata Key:
registry.redis.{cluster}.meta_{ip:port}(Hash,HSETfor key -> value,EXPIREhas the same TTL as the survival key)2. Registration Process
Write the survival key first (without publishing) → Ensure its existence
Then write the metadata hash and set the TTL (set the TTL even without metadata)
Finally, publish the
REGISTERevent to ensure consumers can read the "with metadata" instance on the first read.3. Lookup/Subscribe:
Initial Load:
SCAN registry.redis.{cluster}_*, directly parse{ip:port}from the key name, read the corresponding meta hash, and build... ServiceInstance (with metadata preferred)Subscribe to updates: Upon receiving the REGISTER, first delete the old instance at the same address, then add the new instance (with metadata) to remove duplicates and avoid "coexistence of instances with/without metadata at the same address".
Ⅱ. 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