[
https://issues.apache.org/jira/browse/IGNITE-18351?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Denis Chudov updated IGNITE-18351:
----------------------------------
Description:
Recently, we redesigned the behavior of lock manager, but did not modify the
interface. This leads to the inconsistency between method name and what the
method actually does.
The ticket inspires to improve interface of lock manager to avoid wrong usage
of the interface.
* The lock mode LockMode#NL is used nowhere. It is s candidate for removal.
* The LockManager#release(org.apache.ignite.internal.tx.Lock) method is removed
all locks on a particular transaction, but the method takes a lock (not a
transaction id). Redesign the method in order to release all lock for the
particular transaction.
* The LockManager#release(UUID, LockKey, LockMode) releases a particular lock,
but the method does not take a lock object.
was:
Recently, we redesigned the behavior of lock manager, but did not modify the
interface. This leads the method name does not compatible that the method do.
The ticket inspires to improve interface of lock manager to avoid using the
interface wrong.
* The lock mode LockMode#NL is used nowhere. It is s candidate to remove.
* The LockManager#release(org.apache.ignite.internal.tx.Lock) method is removed
all locks on a particular transaction, but the method takes a lock (not a
transaction id). Redesign the method in order to release all lock for the
particular transaction.
* The LockManager#release(UUID, LockKey, LockMode) releases a particular lock,
but the method does not take a lock object.
> Make a lock menagment interface more friendly
> ---------------------------------------------
>
> Key: IGNITE-18351
> URL: https://issues.apache.org/jira/browse/IGNITE-18351
> Project: Ignite
> Issue Type: Improvement
> Reporter: Vladislav Pyatkov
> Priority: Major
> Labels: ignite-3
>
> Recently, we redesigned the behavior of lock manager, but did not modify the
> interface. This leads to the inconsistency between method name and what the
> method actually does.
> The ticket inspires to improve interface of lock manager to avoid wrong usage
> of the interface.
> * The lock mode LockMode#NL is used nowhere. It is s candidate for removal.
> * The LockManager#release(org.apache.ignite.internal.tx.Lock) method is
> removed all locks on a particular transaction, but the method takes a lock
> (not a transaction id). Redesign the method in order to release all lock for
> the particular transaction.
> * The LockManager#release(UUID, LockKey, LockMode) releases a particular
> lock, but the method does not take a lock object.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
