[jira] [Updated] (TINKERPOP-3228) Update GremlinSessionReuseTxIntegrateTest to test for Graph transaction cleanup

["Stephen Mallette (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/TINKERPOP-3228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stephen Mallette updated TINKERPOP-3228:
----------------------------------------
    Component/s: server

> Update GremlinSessionReuseTxIntegrateTest to test for Graph transaction cleanup
> -------------------------------------------------------------------------------
>
>                 Key: TINKERPOP-3228
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-3228
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 3.7.6
>            Reporter: Ken Hu
>            Priority: Major
>
> The tests in GremlinSessionReuseTxIntegrateTest could improve by having the tests verify that underlying graph in the server doesn't have any open transactions. The tests currently only check that the Session is removed, but in a forced way so the graph itself may have open transactions that need to be rolled back.
> A problem with this, however, is that the Session uses the single threaded executor to run its tasks. This tasks includes closing itself, so in this case, the Session can't check the graph's state during close. This is further complicated by the Graph's Transaction implementation itself. For example, TinkerTransactionGraph determines "openness" on a per thread basis so it can appear that there is no open transaction from one thread but there may be one open from another.
> In summary, find a way in these tests to assert that the underlying Graph actually has all its transactions closed.



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