Skip to content

Add v7-to-v8 migration guide#4541

Merged
uglide merged 8 commits into
masterfrom
im/v7_to_v8-migration-guide
May 27, 2026
Merged

Add v7-to-v8 migration guide#4541
uglide merged 8 commits into
masterfrom
im/v7_to_v8-migration-guide

Conversation

@uglide

@uglide uglide commented May 25, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Documentation-only change; no runtime or library code is modified.

Overview
Adds a Jedis 7.5 → 8.0 migration guide (docs/migration-guides/v7-to-v8.md) and links it in MkDocs under Migrating to newer versions as Jedis 8.

The guide documents breaking changes (RESP3 auto-negotiation by default, TLS hostname verification, removal of JedisPooled/JedisSentineled and UnifiedJedis public constructors, Transaction/ClusterPipeline/CommandObjects/broadcastCommand API changes, deleted sharding utils, commons-pool2 2.13.1), deprecations (JedisCluster, legacy SSL setters, RediSearch cursor APIs), and v8 features (cluster request/response policies, RESP3 push handling, ftAggregateIterator, configGet/configSet, shared pipeline executor, sentinel pool map), with before/after Java snippets.

Reviewed by Cursor Bugbot for commit b3460a5. Bugbot is set up for automated code reviews on this repo. Configure here.

@uglide uglide added this to the 8.0.0 milestone May 25, 2026
@uglide uglide requested a review from atakavci May 25, 2026 07:35
@jit-ci

jit-ci Bot commented May 25, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown

Test Results

  203 files  ±0    203 suites  ±0   9m 53s ⏱️ +12s
7 767 tests ±0  7 705 ✅ +724  62 💤  - 724  0 ❌ ±0 
7 787 runs  ±0  7 725 ✅ +726  62 💤  - 726  0 ❌ ±0 

Results for commit 15498e8. ± Comparison against base commit 9a2dcd5.

♻️ This comment has been updated with latest results.

@uglide uglide requested a review from ggivo May 25, 2026 07:38
@uglide

uglide commented May 25, 2026

Copy link
Copy Markdown
Contributor Author

Compiled html is here https://github.com/redis/jedis/actions/runs/26389452267

Comment thread docs/migration-guides/v7-to-v8.md Outdated
Comment thread docs/migration-guides/v7-to-v8.md Outdated
Comment thread docs/migration-guides/v7-to-v8.md
Comment thread docs/migration-guides/v7-to-v8.md Outdated

@ggivo ggivo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,
Some small findings

Should we also consider adding "TLS hostname verification is now enforced by default " - #4495

Comment thread docs/migration-guides/v7-to-v8.md Outdated
Comment thread docs/migration-guides/v7-to-v8.md Outdated
Comment thread docs/migration-guides/v7-to-v8.md Outdated
@uglide uglide merged commit b617aa0 into master May 27, 2026
15 checks passed
@uglide uglide deleted the im/v7_to_v8-migration-guide branch May 27, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants