Skip to content

Align Jedis restore TTL parameter with long type#3388

Open
DragonFSKY wants to merge 1 commit into
spring-projects:mainfrom
DragonFSKY:gh-3386-jedis-restore-long-ttl
Open

Align Jedis restore TTL parameter with long type#3388
DragonFSKY wants to merge 1 commit into
spring-projects:mainfrom
DragonFSKY:gh-3386-jedis-restore-long-ttl

Conversation

@DragonFSKY

Copy link
Copy Markdown

Closes #3386.

This aligns the Jedis RESTORE command mapping with the long TTL parameter exposed by Spring Data Redis and supported by Jedis.

Changes:

  • Pass ttlInMillis through as long for Jedis restore, including the REPLACE path.
  • Remove the previous Integer.MAX_VALUE guard that was only needed when the value was narrowed to int.
  • Add focused GH-3386 unit coverage for TTL values above Integer.MAX_VALUE.

Verification:

  • ./mvnw -Dtest=JedisConnectionUnitTests test

  • git diff --check

  • You have read the Spring Data contribution guidelines.

  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.

  • You submit test cases (unit or integration tests) that back your changes.

  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 24, 2026
@DragonFSKY DragonFSKY force-pushed the gh-3386-jedis-restore-long-ttl branch 2 times, most recently from d8e01bf to 9d98af1 Compare June 24, 2026 15:41
Signed-off-by: Dongliang Xie <dragonfsky@gmail.com>
@DragonFSKY DragonFSKY force-pushed the gh-3386-jedis-restore-long-ttl branch from 9d98af1 to 9d212f8 Compare June 27, 2026 13:52
@DragonFSKY DragonFSKY marked this pull request as ready for review June 27, 2026 13:53
@mp911de mp911de added type: bug A general bug type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align parameter type for restore command to jedis API

4 participants