Add v6 and v7 migration guides #4315

uglide · 2025-10-09T09:59:55Z

No description provided.

Copilot

Pull Request Overview

This PR adds a comprehensive v7 migration guide and reorganizes the existing migration documentation structure. The migration guide documents breaking changes in Jedis 7.0.0, including removal of deprecated sharding features, base class changes, and introduction of new builder patterns.

Removal of deprecated sharding/sharded features (JedisSharding, ShardedPipeline, etc.)
Base class consolidation (PipelineBase → AbstractPipeline, TransactionBase → AbstractTransaction)
Documentation restructuring with new migration guide structure

Reviewed Changes

Copilot reviewed 25 out of 28 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/test/java/redis/clients/jedis/builders/UnifiedJedisConstructorReflectionTest.java	Updated test to remove reference to deprecated ShardedConnectionProvider
src/test/java/redis/clients/jedis/benchmark/ShardingBenchmark.java	Removed entire benchmark class for deprecated sharding functionality
src/test/java/redis/clients/jedis/benchmark/ShardedBenchmark.java	Removed entire benchmark class for deprecated sharded functionality
src/test/java/redis/clients/jedis/ShardingTest.java	Removed comprehensive test class for deprecated sharding functionality
src/test/java/redis/clients/jedis/ShardedPipelineTest.java	Removed test class for deprecated sharded pipeline functionality
src/test/java/redis/clients/jedis/ShardedConnectionTest.java	Removed test class for deprecated sharded connection functionality
src/main/java/redis/clients/jedis/providers/ShardedConnectionProvider.java	Removed deprecated sharded connection provider class
src/main/java/redis/clients/jedis/mcf/MultiDbTransaction.java	Updated to extend AbstractTransaction instead of deprecated TransactionBase
src/main/java/redis/clients/jedis/mcf/MultiDbPipeline.java	Updated to extend AbstractPipeline instead of deprecated PipelineBase
src/main/java/redis/clients/jedis/UnifiedJedis.java	Removed deprecated constructors and updated return types
src/main/java/redis/clients/jedis/TransactionBase.java	Removed deprecated base class
src/main/java/redis/clients/jedis/Transaction.java	Updated to extend AbstractTransaction instead of deprecated TransactionBase
src/main/java/redis/clients/jedis/ShardedPipeline.java	Removed deprecated sharded pipeline class
src/main/java/redis/clients/jedis/ShardedCommandObjects.java	Removed deprecated sharded command objects class
src/main/java/redis/clients/jedis/ShardedCommandArguments.java	Removed deprecated sharded command arguments class
src/main/java/redis/clients/jedis/ReliableTransaction.java	Updated to extend AbstractTransaction instead of deprecated TransactionBase
src/main/java/redis/clients/jedis/PipelineBase.java	Removed deprecated base class
src/main/java/redis/clients/jedis/Pipeline.java	Updated to extend AbstractPipeline instead of deprecated PipelineBase
src/main/java/redis/clients/jedis/MultiNodePipelineBase.java	Updated to extend AbstractPipeline instead of deprecated PipelineBase
src/main/java/redis/clients/jedis/JedisSharding.java	Removed deprecated sharding client class
mkdocs.yml	Reorganized migration guide structure and added new v7 migration guide
docs/migration-guides/v6-to-v7.md	Added comprehensive migration guide for v6 to v7 changes
docs/migration-guides/v3-to-v4.md	Updated cross-references to use new file paths
docs/failover.md	Added migration section and updated documentation for v7 changes
README.md	Updated version references and simplified getting started section

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

src/main/java/redis/clients/jedis/mcf/MultiDbTransaction.java

docs/migration-guides/v6-to-v7.md

- Bump client version - Update list of supported Redis versions - Remove references to JedisPool - Remove reference to Google Group

Copilot

Pull Request Overview

Copilot reviewed 4 out of 7 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

docs/migration-guides/v6-to-v7.md:1

Extra closing parenthesis after the link.

# Jedis 7.0.0 Migration Guide

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

README.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

github-actions · 2025-10-09T10:19:13Z

Test Results

280 files 280 suites 11m 53s ⏱️
10 063 tests 9 007 ✅ 1 056 💤 0 ❌
2 701 runs 2 701 ✅ 0 💤 0 ❌

Results for commit d312f2b.

♻️ This comment has been updated with latest results.

Copilot

Pull Request Overview

Copilot reviewed 4 out of 7 changed files in this pull request and generated 1 comment.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

docs/migration-guides/v6-to-v7.md

…im/6-to-7-migration-guide

Copilot

Pull Request Overview

Copilot reviewed 5 out of 8 changed files in this pull request and generated no new comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

@tag

commit 297279e Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 13:06:10 2025 +0200 Add v6 and v7 migration guides (#4315) * Add migration guide for v7 * Move all migration guides to subfolder * Update readme: - Bump client version - Update list of supported Redis versions - Remove references to JedisPool - Remove reference to Google Group * Remove duplicated content in the migration guide * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add missing v5 to v6 migration guide * Fix broken link * Update outdated modules section in README --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> commit 3645601 Author: Ivo Gaydazhiev <ivo.gaydazhiev@redis.com> Date: Thu Oct 9 13:53:02 2025 +0300 Dedicated profile for running Scenario tests (#4312) * Dedicated profile for running Scenario tests to run only scenario tests mvn -Pscenario-tests clean verify * format * remove import after rebase conflict * fix RedisRestAPIIT scenario test - increase expected bdb's count to account for additional db added to test env - fix incorrect bdbid used for lag-aware availablity checks now using bdb of'"re-active-active" * fix LagAwareStrategySslIT scenario test - Now exception is propagated in case of ssl connection errors instead of returning UNHEALTHY - updated the test to current expectations - Enhanced test to ensure untrusted default certificate * java 8 compatibility * remove @tag("integration") from IntegrationTest* to avoid multiple runs of same test --------- Co-authored-by: Igor Malinovskiy <u.glide@gmail.com> commit 07bf3b0 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 12:00:36 2025 +0200 [automatic failover] Update failover docs (#4314) Update failover docs - Add migration guide from 6.x to 7.0 - Add instructions on optional deps - Clean up wording to refer to multiDbClient instead of connection provider commit 158e726 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 11:14:28 2025 +0200 Remove deprecated constructors, classes and JedisSharding (#4311) * Remove JedisSharding * Remove deprecated constructors in UnifiedJedis * Clean up UnifiedJedisConstructorReflectionTest * Remove remaining ShardedCommandArguments * Remove deprecated PipelineBase * Remove deprecated TransactionBase * Remove unused attr in MultiDbTransaction commit 5db1a39 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 10:04:06 2025 +0200 Remove spellcheck (#4309) Nowadays, AI reviewing tools like CoPilot catch spellcheck issues better than pyspelling without a burden of maintaining wordlist.txt

@tag

…4317) * [automatic failover] Replace EchoStrategy with PingStrategy (#4313) * - replace EchoStrategy with PingStrategy * - fix doc * [automatic failover] Remove UnifiedJedis experimental constructor accepting MultiDbConnectionProvider (#4316) * Remove UnifiedJedis(MultiDbConnectionProvider) experimental constructor The recommended way to create MultiDbClient is using MultiDbClient.builder(). Update all tests to use the builder pattern. Fixes #4307 * revert unintentional change in MultiDbClientBuilder * Squashed commit of the following: commit 297279e Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 13:06:10 2025 +0200 Add v6 and v7 migration guides (#4315) * Add migration guide for v7 * Move all migration guides to subfolder * Update readme: - Bump client version - Update list of supported Redis versions - Remove references to JedisPool - Remove reference to Google Group * Remove duplicated content in the migration guide * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add missing v5 to v6 migration guide * Fix broken link * Update outdated modules section in README --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> commit 3645601 Author: Ivo Gaydazhiev <ivo.gaydazhiev@redis.com> Date: Thu Oct 9 13:53:02 2025 +0300 Dedicated profile for running Scenario tests (#4312) * Dedicated profile for running Scenario tests to run only scenario tests mvn -Pscenario-tests clean verify * format * remove import after rebase conflict * fix RedisRestAPIIT scenario test - increase expected bdb's count to account for additional db added to test env - fix incorrect bdbid used for lag-aware availablity checks now using bdb of'"re-active-active" * fix LagAwareStrategySslIT scenario test - Now exception is propagated in case of ssl connection errors instead of returning UNHEALTHY - updated the test to current expectations - Enhanced test to ensure untrusted default certificate * java 8 compatibility * remove @tag("integration") from IntegrationTest* to avoid multiple runs of same test --------- Co-authored-by: Igor Malinovskiy <u.glide@gmail.com> commit 07bf3b0 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 12:00:36 2025 +0200 [automatic failover] Update failover docs (#4314) Update failover docs - Add migration guide from 6.x to 7.0 - Add instructions on optional deps - Clean up wording to refer to multiDbClient instead of connection provider commit 158e726 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 11:14:28 2025 +0200 Remove deprecated constructors, classes and JedisSharding (#4311) * Remove JedisSharding * Remove deprecated constructors in UnifiedJedis * Clean up UnifiedJedisConstructorReflectionTest * Remove remaining ShardedCommandArguments * Remove deprecated PipelineBase * Remove deprecated TransactionBase * Remove unused attr in MultiDbTransaction commit 5db1a39 Author: Igor Malinovskiy <u.glide@gmail.com> Date: Thu Oct 9 10:04:06 2025 +0200 Remove spellcheck (#4309) Nowadays, AI reviewing tools like CoPilot catch spellcheck issues better than pyspelling without a burden of maintaining wordlist.txt --------- Co-authored-by: Ivo Gaydazhiev <ivo.gaydazhiev@redis.com>

uglide requested review from atakavci, Copilot and ggivo October 9, 2025 09:59

Copilot AI reviewed Oct 9, 2025

View reviewed changes

src/main/java/redis/clients/jedis/mcf/MultiDbTransaction.java Show resolved Hide resolved

docs/migration-guides/v6-to-v7.md Outdated Show resolved Hide resolved

uglide added 4 commits October 9, 2025 12:01

Add migration guide for v7

cd80c81

Move all migration guides to subfolder

5340fb6

Update readme:

51c61f0

- Bump client version - Update list of supported Redis versions - Remove references to JedisPool - Remove reference to Google Group

Remove duplicated content in the migration guide

646a37c

uglide force-pushed the im/6-to-7-migration-guide branch from e2e493d to 646a37c Compare October 9, 2025 10:01

uglide requested a review from Copilot October 9, 2025 10:03

Copilot AI reviewed Oct 9, 2025

View reviewed changes

README.md Outdated Show resolved Hide resolved

Update README.md

d312f2b

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Add missing v5 to v6 migration guide

2e2c399

uglide changed the title ~~Add v7 migration guide~~ Add v6 and v7 migration guides Oct 9, 2025

uglide requested a review from Copilot October 9, 2025 10:43

Copilot AI reviewed Oct 9, 2025

View reviewed changes

docs/migration-guides/v6-to-v7.md Outdated Show resolved Hide resolved

uglide added 2 commits October 9, 2025 12:44

Merge remote-tracking branch 'origin/im/6-to-7-migration-guide' into …

f24d4cf

…im/6-to-7-migration-guide

Fix broken link

cd648ad

uglide requested a review from Copilot October 9, 2025 10:48

Copilot AI reviewed Oct 9, 2025

View reviewed changes

uglide and others added 2 commits October 9, 2025 12:55

Update outdated modules section in README

75590c7

Merge branch 'master' into im/6-to-7-migration-guide

3065764

uglide merged commit 297279e into master Oct 9, 2025
10 checks passed

uglide deleted the im/6-to-7-migration-guide branch October 9, 2025 11:06

ggivo added the documentation label Oct 10, 2025

ggivo added this to the 7.0.0 milestone Oct 10, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add v6 and v7 migration guides #4315

Add v6 and v7 migration guides #4315

Uh oh!

uglide commented Oct 9, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

github-actions bot commented Oct 9, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Add v6 and v7 migration guides #4315

Add v6 and v7 migration guides #4315

Uh oh!

Conversation

uglide commented Oct 9, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

github-actions bot commented Oct 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

github-actions bot commented Oct 9, 2025 •

edited

Loading