[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786121#comment-17786121 ] ASF GitHub Bot commented on TINKERPOP-3016: --- kenhuuu commented on PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#issuecomment-1811620721 VOTE +1 > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786125#comment-17786125 ] ASF GitHub Bot commented on TINKERPOP-3016: --- kenhuuu commented on PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#issuecomment-1811638396 Needs a CHANGELOG. > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786126#comment-17786126 ] ASF GitHub Bot commented on TINKERPOP-3016: --- kenhuuu commented on code in PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#discussion_r1393504432 ## tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerTransactionGraph.java: ## @@ -72,8 +72,8 @@ public final class TinkerTransactionGraph extends AbstractTinkerGraph { private final TinkerTransaction transaction = new TinkerTransaction(this); -protected Map> vertices = new ConcurrentHashMap<>(); -protected Map> edges = new ConcurrentHashMap<>(); +final Map> vertices = new ConcurrentHashMap<>(); Review Comment: Nit: explicitly state what the access modifer should be. > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786127#comment-17786127 ] ASF GitHub Bot commented on TINKERPOP-3016: --- kenhuuu commented on code in PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#discussion_r1393504695 ## tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerElementContainer.java: ## @@ -221,10 +235,11 @@ public boolean canBeRemoved() { /** * Cleanup changes made in the current transaction. */ -private void reset() { +void reset() { Review Comment: Should this be public? ## tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerElementContainer.java: ## @@ -221,10 +235,11 @@ public boolean canBeRemoved() { /** * Cleanup changes made in the current transaction. */ -private void reset() { +void reset() { Review Comment: Nit: Should this be public? > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786464#comment-17786464 ] ASF GitHub Bot commented on TINKERPOP-3016: --- Cole-Greer commented on PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#issuecomment-1813071423 VOTE +1 > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786528#comment-17786528 ] ASF GitHub Bot commented on TINKERPOP-3016: --- vkagamlyk commented on PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343#issuecomment-1813371071 VOTE+1 > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (TINKERPOP-3016) TinkerTransactionGraph can incorrectly handle some read operations.
[ https://issues.apache.org/jira/browse/TINKERPOP-3016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786529#comment-17786529 ] ASF GitHub Bot commented on TINKERPOP-3016: --- vkagamlyk merged PR #2343: URL: https://github.com/apache/tinkerpop/pull/2343 > TinkerTransactionGraph can incorrectly handle some read operations. > --- > > Key: TINKERPOP-3016 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3016 > Project: TinkerPop > Issue Type: Bug > Components: tinkergraph >Affects Versions: 3.7.0 >Reporter: Valentyn Kahamlyk >Assignee: Valentyn Kahamlyk >Priority: Critical > > With some combination of reading operations, `TinkerTransactionGraph` may > return not the current Element value, but the cached one. > This may also cause update operations to fail with `Conflict: element > modified in another transaction` exception. -- This message was sent by Atlassian Jira (v8.20.10#820010)