[TINKERPOP-1830] Race condition in Tinkergraph index creation

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.0, 3.2.6
• Fix Version/s: 3.2.7, 3.3.1
• Component/s: tinkergraph
• Assignee: Michael Pollmeier
• Reporter: Michael Pollmeier
• Created: 10/Nov/17 01:12
• Updated: 14/Nov/17 12:01
• Resolved: 14/Nov/17 12:01

Description

My colleage Fabian Yamaguchi <[e-mail]> discovered a race condition in tinkergraph's index creation. He fixed it by simply replacing `parallelStream` with `stream`. Quoting his analysis:

> So, reading the code, you see that this.put is called in parallel, but that method seems to contain a race as get is called on the index, checked for null, and a subsequent write is performed. It still seems like using stream here fixes the problem we've been seeing, and the performance hit is not significant.

Issue Links

• links to: GitHub Pull Request #745