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

Jean-Noel Rouvignac
04.59.2015 f6c48ea788101c0356d2a7218351696048658dde
refs
author Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Monday, May 4, 2015 14:59 +0200
committer Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Monday, May 4, 2015 14:59 +0200
commitf6c48ea788101c0356d2a7218351696048658dde
tree d812705909770c1f6e5ccd8c83da69eff4eeda26 tree | zip | gz
parent 023d6304e6c90f955334955aec93d8c9ecef99ba view | diff
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