-
Notifications
You must be signed in to change notification settings - Fork 29
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
Java: Update transaction handling and IT #1284
Java: Update transaction handling and IT #1284
Conversation
* Ease transaction IT updating and fixing. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Fix test and command manager. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * SPOTLESS I HATE YOU Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Copy from #165. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Revert some refactoring. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Fix IT. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Revert timeout change. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * SPOTLESS I HATE YOU Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Group commands in transaction IT. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Refresh Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Test fixes. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> * Fix for redis 6. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com> --------- Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
…ansaction_IT Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
…ansaction_IT Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
@Yury-Fridlyand any updates? CI is failing |
Fixed |
java/integTest/src/test/java/glide/TransactionTestUtilities.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
@shohamazon Can you please review? |
…ansaction_IT Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
CI failure will be fixed by merging amazon-contributing/redis-rs#143 |
.pexpire(genericKey1, 42, ExpireOptions.NEW_EXPIRY_GREATER_THAN_CURRENT) | ||
.pexpireAt(genericKey1, 42, ExpireOptions.HAS_NO_EXPIRY); | ||
} | ||
// TODO add BZMPOP from #194 here |
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.
why? its a sorted set command
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.
oh, it is copy-paste artifact
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.
nice work!
be sure to not merge before CI is green :) |
java/integTest/src/test/java/glide/TransactionTestUtilities.java
Outdated
Show resolved
Hide resolved
…ansaction_IT Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
* Update transaction handling and IT (#160) * Ease transaction IT updating and fixing. Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Issue #, if available:
N/A
Description of changes:
CommandManager
transaction related methods.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.