search

marklogic / marklogic-jena

Adapter for using MarkLogic with the Jena RDF Framework
Other
5 stars 11 forks source link

Concurrent updates and queries do not all execute. Issue #62 #63

Closed grechaw closed 7 years ago

grechaw commented 7 years ago

I'd like QA and another dev to review this fix.

QA, I'd like to add a test for covering this, and have written something, but I think it belongs in functional rather than unit tests, since I found it as a multithreading issue.

Actually, I think I could reproduce the issue in a single-threaded test too, so I"ll try that first and add a commit with the test in it.

grechaw commented 7 years ago

I was able indeed to write a test for this issue.

Since this bug means that the existing marklogic-jena is not compatible with 9.0-3, this will be important to drive a release in the fall that depends on a new java client and server.

grechaw commented 7 years ago

Review requested -- @akshaysonvane , see what you think of this change. I verified that the synchronization solves the threading issue, maybe you'll consider some way I'd not thought of.