-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pipeline & Transaction with failover to multi cluster #3602
Conversation
to replace 'Base' classes; as the base classes don't have any implementation anymore and so the word abstract better represents those classes.
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #3602 +/- ##
============================================
+ Coverage 71.60% 75.33% +3.73%
- Complexity 4883 4890 +7
============================================
Files 290 297 +7
Lines 15610 14963 -647
Branches 1115 1130 +15
============================================
+ Hits 11178 11273 +95
+ Misses 3946 3194 -752
- Partials 486 496 +10 ☔ View full report in Codecov by Sentry. |
} | ||
|
||
public Transaction multi() { | ||
public AbstractTransaction multi() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
src/test/java/redis/clients/jedis/modules/graph/GraphTransactionTest.java
Show resolved
Hide resolved
src/test/java/redis/clients/jedis/misc/AutomaticFailoverTest.java
Outdated
Show resolved
Hide resolved
src/main/java/redis/clients/jedis/activeactive/CircuitBreakerCommandExecutor.java
Show resolved
Hide resolved
src/main/java/redis/clients/jedis/activeactive/MultiClusterPipeline.java
Show resolved
Hide resolved
a3d2fb2
to
757c5f0
Compare
@sazzad16 in version 5.1.0 Transaction class no longer supported publish ? |
@qyi71 No. |
right |
Resolves #3568
closes #3594