redis / lettuce

Advanced Java Redis client for thread-safe sync, async, and reactive usage. Supports Cluster, Sentinel, Pipelining, and codecs.
https://lettuce.io
MIT License
5.3k stars 949 forks source link

Add support for `SPUBLISH` #2838

Closed atakavci closed 2 months ago

atakavci commented 2 months ago

Closes/Fixes #2757

implementation is similar to PUBLISH. using existing routing mechanism. some of commented/disabled tests related to SSUBSCRIBE are enabled regarding to TODO notes.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 63.33333% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 77.64%. Comparing base (43843bf) to head (b906da7). Report is 252 commits behind head on main.

Files Patch % Lines
...io/lettuce/core/cluster/PubSubClusterEndpoint.java 60.00% 2 Missing :warning:
...ore/cluster/pubsub/RedisClusterPubSubListener.java 0.00% 2 Missing :warning:
...va/io/lettuce/core/pubsub/RedisPubSubListener.java 0.00% 2 Missing :warning:
...io/lettuce/core/AbstractRedisReactiveCommands.java 0.00% 1 Missing :warning:
...core/cluster/pubsub/RedisClusterPubSubAdapter.java 0.00% 1 Missing :warning:
...ava/io/lettuce/core/pubsub/RedisPubSubAdapter.java 0.00% 1 Missing :warning:
...e/core/pubsub/RedisPubSubReactiveCommandsImpl.java 0.00% 1 Missing :warning:
.../api/coroutines/BaseRedisCoroutinesCommandsImpl.kt 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2838 +/- ## ============================================ - Coverage 78.71% 77.64% -1.07% - Complexity 6786 7238 +452 ============================================ Files 508 539 +31 Lines 22765 24519 +1754 Branches 2446 2607 +161 ============================================ + Hits 17919 19039 +1120 - Misses 3717 4277 +560 - Partials 1129 1203 +74 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.