Skip to content

Add releaseLock overload that accepts a value#547

Merged
sleberknight merged 3 commits into
mainfrom
add-release-lock-with-value-overload
Mar 13, 2026
Merged

Add releaseLock overload that accepts a value#547
sleberknight merged 3 commits into
mainfrom
add-release-lock-with-value-overload

Conversation

@sleberknight

@sleberknight sleberknight commented Mar 13, 2026

Copy link
Copy Markdown
Member

Add releaseLock(key, value, sessionId) to mirror the existing acquireLock(key, value, session) overload. The new overload allows callers to store application-specific data in the key at the same time as releasing the lock (e.g. information about who released it).

The existing releaseLock(key, sessionId) now delegates to the new overload with an empty string value, preserving backward compatibility.

Add releaseLock(key, value, sessionId) to mirror the existing
acquireLock(key, value, session) overload. The new overload allows
callers to store application-specific data in the key at the same
time as releasing the lock.

The existing releaseLock(key, sessionId) now delegates to the new
overload with an empty string value, preserving backward compatibility.

Add integration test acquireAndReleaseLockWithValue to verify that the
value is stored correctly when releasing the lock.

Closes #416

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@sleberknight sleberknight self-assigned this Mar 13, 2026
@sleberknight sleberknight added the enhancement A request for change or improvement to an existing feature label Mar 13, 2026
@sleberknight sleberknight added this to the 1.11.0 milestone Mar 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

@sleberknight
sleberknight merged commit 6c67da2 into main Mar 13, 2026
13 checks passed
@sleberknight
sleberknight deleted the add-release-lock-with-value-overload branch March 13, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement A request for change or improvement to an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant