stephen mallette created TINKERPOP-1829:
---
Summary: Improve flexibility of detachment for EventStrategy
Key: TINKERPOP-1829
URL: https://issues.apache.org/jira/browse/TINKERPOP-1829
Project:
[
https://issues.apache.org/jira/browse/TINKERPOP-1807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246510#comment-16246510
]
ASF GitHub Bot commented on TINKERPOP-1807:
---
Github user spmallette commented on the issue:
GitHub user mpollmeier opened a pull request:
https://github.com/apache/tinkerpop/pull/745
TINKERPOP-1830: fix race condition in TinkerIndex
My colleage @fabsx00 discovered a race condition in tinkergraph's index
creation. He fixed it by simply replacing `parallelStream` with
Michael Pollmeier created TINKERPOP-1830:
Summary: Race condition in Tinkergraph index creation
Key: TINKERPOP-1830
URL: https://issues.apache.org/jira/browse/TINKERPOP-1830
Project:
[
https://issues.apache.org/jira/browse/TINKERPOP-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246860#comment-16246860
]
ASF GitHub Bot commented on TINKERPOP-1830:
---
GitHub user mpollmeier opened a pull request:
[
https://issues.apache.org/jira/browse/TINKERPOP-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247033#comment-16247033
]
ASF GitHub Bot commented on TINKERPOP-1830:
---
Github user mpollmeier commented on the issue:
Github user robertdale commented on the issue:
https://github.com/apache/tinkerpop/pull/745
The race condition is in `put()` and it's used elsewhere. It would make
more sense to fix `put()` to be thread-safe. Then we can keep
`parallelStream()`
```
Map
[
https://issues.apache.org/jira/browse/TINKERPOP-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246988#comment-16246988
]
ASF GitHub Bot commented on TINKERPOP-1830:
---
Github user robertdale commented on the issue:
Github user mpollmeier commented on the issue:
https://github.com/apache/tinkerpop/pull/745
Agreed, that's a better solution @robertdale. Note that `putIfAbsent` is
not generally thread safe, but since we use `ConcurrentHashMap` it should be.
I'll amend the commit.
---