Skip to content

Upgrade to Jedis 4.2 #2287

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

Closed
wants to merge 7 commits into from
Closed

Upgrade to Jedis 4.2 #2287

wants to merge 7 commits into from

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Mar 25, 2022

Upgrade to Jedis 4. This upgrade introduces breaking changes as several Jedis types were removed (e.g. JedisShardInfo) or relocated to packages (e.g. response and parameter types).

It is no longer possible to use pipelining and transactions together.

Closes #2212

@mp911de mp911de added in: jedis Jedis driver type: enhancement A general enhancement labels Mar 25, 2022
mp911de added 6 commits March 25, 2022 11:08
Add support for GEOSEARCH, GEOSEARCHSTORE, XCLAIMJUSTID, and XPENDING summary.
Reformat code. Reduce visibility, simplify code.
Also, refine docs regarding driver ordering.
Consistently use InvalidDataAccessApiUsageException to signal that an operation is not allowed/supported in the current mode of operation.
Replace converter objects with conversion within the actual method to remove indirections.
@mp911de mp911de changed the title Upgrade to Jedis 4 Upgrade to Jedis 4.2 Mar 25, 2022
@mp911de mp911de requested a review from christophstrobl March 25, 2022 10:18
@mp911de
Copy link
Member Author

mp911de commented Apr 20, 2022

Paging @gkorland

@RajKannanIyyappan
Copy link

Thanks @mp911de for the PR. @christophstrobl Can you please review?

christophstrobl pushed a commit that referenced this pull request May 18, 2022
The upgrade also allows to enable support for GEOSEARCH, GEOSEARCHSTORE, XCLAIMJUSTID and XPENDING.

Closes: #2212
Original Pull Request: #2287
christophstrobl pushed a commit that referenced this pull request May 18, 2022
Reformat code. Reduce visibility, simplify code.

Original Pull Request: #2287
christophstrobl pushed a commit that referenced this pull request May 18, 2022
Also, refine docs regarding driver ordering.

Original Pull Request: #2287
christophstrobl pushed a commit that referenced this pull request May 18, 2022
Consistently use InvalidDataAccessApiUsageException to signal that an operation is not allowed/supported in the current mode of operation.
Replace converter objects with conversion within the actual method to remove indirections.

Original Pull Request: #2287
christophstrobl added a commit that referenced this pull request May 18, 2022
@christophstrobl christophstrobl deleted the issue/3.0.x/2212 branch May 18, 2022 11:45
@sazzad16
Copy link
Contributor

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: jedis Jedis driver type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to Jedis 4.2
4 participants