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

Jean-Noel Rouvignac
06.57.2014 9ccf0cb56369e9ad8a60ff30e42575780a50f1e7
refs
author Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Thursday, November 6, 2014 16:57 +0100
committer Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Thursday, November 6, 2014 16:57 +0100
commit9ccf0cb56369e9ad8a60ff30e42575780a50f1e7
tree e9d6285270ffbf5e31bee077507adb82b1c018e6 tree | zip | gz
parent 4774f6dff8259690bbef9858ada3f09fcf4e0ead view | diff
OPENDJ-1623 NullPointerException while doing psearch (with changesonly=false) on cn=changelog

Problem introduced by r11087.

ChangelogBackend.java:
In initializePersistentSearch(), always create the ChangeNumberRange object.
Problem is that with changesOnly=false, this object will be created twice: one from ChangelogBackend.registerPersistentSearch() and another from ChangelogBackend.search().
I really think we should have a unique Backend.preSearch() method which is always called regardless of whether this is a persistent search or not.
This would allow to factorize search initialization in a single place, before it can be used to return entries by the initial and persistent phases.
1 files modified
17 ■■■■■ changed files
opendj-sdk/opendj3-server-dev/src/server/org/opends/server/backends/ChangelogBackend.java 17 ●●●●● diff | view | raw | blame | history