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

Jean-Noel Rouvignac
04.59.2015 2ed3ee723fab71228c360e58b22b26e870778341
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
commit2ed3ee723fab71228c360e58b22b26e870778341
tree 3f03bc282dfd947dd0d2d06b5a158fc1f4887545 tree | zip | gz
parent 858b066d8866b751443d0804600e68744b7d42f6 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-server-legacy/src/main/java/org/opends/server/backends/persistit/PersistItStorage.java 15 ●●●●● diff | view | raw | blame | history