Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Concurrency support #15

Closed
tdauth opened this issue Feb 5, 2019 · 2 comments
Closed

Concurrency support #15

tdauth opened this issue Feb 5, 2019 · 2 comments
Assignees
@tdauth
Labels
enhancement New feature or request

Comments

@tdauth
Copy link
Contributor

tdauth commented Feb 5, 2019

The NFA might be updated concurrently in the future. Add concurrency support to protect critical section from data races.
@tdauth tdauth added the enhancement New feature or request label Feb 5, 2019
@tdauth tdauth self-assigned this Feb 5, 2019
@beatngu13
Copy link
Contributor

As a side note: As soon as we switch the persistence to a database (be it a RDBMS, a Graph DB, etc.), we probably get concurrency support for free due to transactions.

tdauth added a commit that referenced this issue Feb 27, 2019
@tdauth
Initial concurrency support #15
3c67139 
Basic concurrency support with synchronized.
@tdauth
Copy link
Contributor Author

tdauth commented Apr 1, 2019

I will close this issue since I have added synchronized wherever concurrent access could happen.
It is not the most performant way but the easiest. STM or CAS would be too complicated.
When we use Neo4J, this is not required anymore.

@tdauth tdauth closed this as completed Apr 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tdauth tdauth

Labels
enhancement New feature or request
Milestone
No milestone
Development

No branches or pull requests
2 participants
@beatngu13 @tdauth