Skip to content

Conversation

@christophstrobl
Copy link
Member

We now support EX/PX and NX/XX arguments along with the SET command for both xetorthio/jedis and mp911de/lettuce.

Jedis 2.8 does not support all combinations of EX/PX and NX/XX. To work around those limitations we delegate to the according set methods to execute related commands or fail fast if there is no way for delegation.

We now support EX/PX and NX/XX arguments along with the SET command for both xetorthio/jedis and mp911de/lettuce. 

Jedis 2.8 does not support all combinations of EX/PX and NX/XX. To work around those limitations we delegate to the according set methods to execute related commands or fail fast if there is no way for delegation.
@christophstrobl christophstrobl changed the title DATAREDIS-316 - Add EX/PX, NX/ options to SET command. DATAREDIS-316 - Add EX/PX, NX/XX options to SET command. Feb 16, 2016
Extend JavaDoc documentation. Spelling fixes. Add Expiration.from factory method to create Expiration from all available time units.
@mp911de mp911de force-pushed the issue/DATAREDIS-316 branch from 7855daf to 57b1c55 Compare February 16, 2016 13:58
mp911de pushed a commit that referenced this pull request Feb 16, 2016
We now support EX/PX and NX/XX arguments along with the SET command for both xetorthio/jedis and mp911de/lettuce.

Jedis 2.8 does not support all combinations of EX/PX and NX/XX. To work around those limitations we delegate to the according set methods to execute related commands or fail fast if there is no way for delegation.

Original pull request: #170.
mp911de added a commit that referenced this pull request Feb 16, 2016
Extend JavaDoc documentation. Spelling fixes. Add Expiration.from factory method to create Expiration from all available time units.

Original pull request: #170.
@mp911de
Copy link
Member

mp911de commented Feb 16, 2016

Merged with 1151bcf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants