mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Jean-Noel Rouvignac
05.11.2015 a2170aa5a2537386143e485d4b5e8788985b91d3
refs
author Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Tuesday, May 5, 2015 14:11 +0200
committer Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Tuesday, May 5, 2015 14:11 +0200
commita2170aa5a2537386143e485d4b5e8788985b91d3
tree 5dace964710b8eb1c72ac47135720312b6203b9c tree | zip | gz
parent f6517225beff9945f29165968b838882cc1f4a81 view | diff
OPENDJ-1994 Setup with persistit backend containing automatically generated data hangs

Reapply r12193 after https://stash.forgerock.org/projects/COMMONS/repos/forgerock-persistit/commits/85271bef2cd4b154d923d6b775bd5fdaa59f0b1b .

Original commit message was:





Fix continuous integration build failure:

Failed Test: org.opends.server.backends.pluggable.persistit.PersistitTestCase#testImportLDIF
Failure Cause: org.opends.server.types.DirectoryException: IllegalStateException: Database is already open, either the backend is enabled or an import is currently running. (PersistItStorage.java:653 PersistItStorage.java:643 TracedStorage.java:276 Importer.java:1161 Importer.java:1070 Importer.java:905 RootContainer.java:321 RootContainer.java:185 BackendImpl.java:676 PluggableBackendImplTestCase.java:848 NativeMethodAccessorImpl.java:-2 NativeMethodAccessorImpl.java:57 DelegatingMethodAccessorImpl.java:43 Method.java:606 MethodInvocationHelper.java:76 Invoker.java:673 Invoker.java:846 Invoker.java:1170 TestMethodWorker.java:125 TestMethodWorker.java:109 TestRunner.java:1147 ...)
org.opends.server.backends.pluggable.RootContainer.importLDIFWithOnDiskMerge(RootContainer.java:336)
org.opends.server.backends.pluggable.RootContainer.importLDIF(RootContainer.java:185)
org.opends.server.backends.pluggable.BackendImpl.importLDIF(BackendImpl.java:676)
org.opends.server.backends.pluggable.PluggableBackendImplTestCase.testImportLDIF(PluggableBackendImplTestCase.java:848)

The root cause of this problem is that PersisitIt refuses to release exchanges from current thread if it is not the thread that acquired that exchange from the pool.
The fix is to not release the exchanges at all as per http://sources.forgerock.org/cru/CR-6815#c73584 .


PersistItStorage.java:
In ImporterImpl:
- removed the field allExchanges
- in close(), do not release any exchanges since we are about to close the PersistIt instance
1 files modified
15 ■■■■■ changed files
opendj-sdk/opendj-server-legacy/src/main/java/org/opends/server/backends/persistit/PersistItStorage.java 15 ●●●●● diff | view | raw | blame | history