Conversation
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (postgres storage)
|
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the Valkey storage implementation from regular Redis pub/sub to sharded pub/sub (SPUBLISH/SSUBSCRIBE). The change uses Redis hash tags {client} in channel names to ensure all client channels are routed to the same shard in a Redis cluster, which is a requirement for sharded pub/sub to work correctly.
- Updated channel naming from
client:to{client}:format with hash tags - Migrated pub/sub methods to use sharded variants (SPublish, SSubscribe, SUnsubscribe)
- Updated channel parsing logic to handle the new format
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
Performance Metrics (postgres storage)
📊 Performance Metrics (dnsmasq storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (cluster-valkey storage) ❌ Bridge service not accessible after 30 seconds 📊 Performance Metrics (nginx storage) ❌ Bridge service not accessible after 30 seconds |
📊 Performance MetricsPerformance Metrics (memory storage)
|
📊 Performance MetricsPerformance Metrics (postgres storage)
|
📊 Performance MetricsPerformance Metrics (cluster-valkey storage)
|
📊 Performance MetricsPerformance Metrics (dnsmasq storage)
|
📊 Performance MetricsPerformance Metrics (nginx storage)
|

Setup Cluster Valkey, bridge. In 2 terminals:
Terminal 1: sending messages:
Terminal 2: receiving messeges:
All messages received.