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

Alexei Scherbakov commented on IGNITE-12209:
--------------------------------------------

[~nizhikov]

Note what 
org.apache.ignite.internal.processors.cache.transactions.IgniteTxState#allEntries
 and 
org.apache.ignite.internal.processors.cache.transactions.IgniteTxState#cacheIds 
are unsynchronized and can be concurrently updated if a transaction enlists 
keys in the moment of view producing.

So current implementation is unsafe but probably will work somehow. 
I suggest to enclose methods in try .. catch(Throwable) to implement fallback 
in case something goes wrong.


> Transaction system view
> -----------------------
>
>                 Key: IGNITE-12209
>                 URL: https://issues.apache.org/jira/browse/IGNITE-12209
>             Project: Ignite
>          Issue Type: Sub-task
>    Affects Versions: 2.7.6
>            Reporter: Nikolay Izhikov
>            Assignee: Nikolay Izhikov
>            Priority: Major
>              Labels: IEP-35
>             Fix For: 2.8
>
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
>
> IGNITE-12145 finished
> We should add transactions to the system views.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to