I think the in vm case is only used in some tests, so it is not critical but would be a nice addition to have it correct.
On 7 April 2015 at 22:33, Clebert Suconic <[email protected]> wrote: > I took this JIRA to fix here: > > https://issues.apache.org/jira/browse/AMQ-5709 > > https://github.com/apache/activemq/pull/82 > > > > and I discovered a race into LockFile.java > > > Basically Lock will return false if you run the test I wrote in loop. > It's non related to my change.. but I will add the test anyways... the > test could fail eventually in loop > > > locked is not synchronized in all paths.. There are some issues with it. > > > Should I leave this alone? It seems it's only an issue under the same > VM though. although it could be something else happening.
