[ 
https://issues.apache.org/jira/browse/IGNITE-22917?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17873615#comment-17873615
 ] 

Anton Vinogradov edited comment on IGNITE-22917 at 8/16/24 11:52 AM:
---------------------------------------------------------------------

It seems we've lost near cache tx support, but even before first version was 
released. 
Should we reimplement it or get rid of it?


was (Author: av):
It seem's we've lost near cache tx support, but even before first version was 
released. 
Should we reimplement it or get rid of it?

> Remove GridDhtLockRequest#nearKeys
> ----------------------------------
>
>                 Key: IGNITE-22917
>                 URL: https://issues.apache.org/jira/browse/IGNITE-22917
>             Project: Ignite
>          Issue Type: Sub-task
>            Reporter: Ilya Shishkov
>            Assignee: Ilya Shishkov
>            Priority: Minor
>              Labels: ise
>             Fix For: 2.17
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> {{GridDhtLockRequest#addNearKey(KeyCacheObject)}} does not have any usage, 
> i.e. {{GridDhtLockRequest#nearKeys}} is not updated at all. 
> So, {{nearKeys}} always returns empty collection and, moreover, some code 
> becomes useless, eg. {{GridNearTransactionalCache#startRemoteTx(UUID, 
> GridDhtLockRequest)}} [1], etc.
> We have to figure out whether this is redundant code or whether we have 
> *_lost some useful functionality_*.
> # 
> https://github.com/apache/ignite/blob/f4654f98607648252c5164c0fd1a81eff279ca26/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTransactionalCache.java#L285



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to