This is an automated email from the git hooks/post-receive script.
lkrispen pushed a change to branch master
in repository 389-ds-base.
from fa71a0a Ticket 49278 - GetEffectiveRights gives false-negative
new 339caec Ticket 49446 - cleanallruv should ignore cleaned replica Id in
processing changelog if in force mode
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.
Summary of changes:
ldap/servers/plugins/replication/repl5_replica_config.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
Show replies by date
This is an automated email from the git hooks/post-receive script.
lkrispen pushed a commit to branch master
in repository 389-ds-base.
commit 339caec282be57e488d553939b9837e38f0a9410
Author: Ludwig Krispenz <lkrispen(a)redhat.com>
Date: Thu Jan 11 15:56:21 2018 +0100
Ticket 49446 - cleanallruv should ignore cleaned replica Id in processing changelog if
in force mode
Bug: If the startcsn is calculated based on a cleaned rid, it could be missing from
the changelog.
Fix: In force mode we do not care that the topology gets in sync for the cleaned RID,
so we can ignore it
in an earlier stage, instead of setting it to precleane only.
Reviewed by: Thierry, thanks
---
ldap/servers/plugins/replication/repl5_replica_config.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/ldap/servers/plugins/replication/repl5_replica_config.c
b/ldap/servers/plugins/replication/repl5_replica_config.c
index e025f34..005528a 100644
--- a/ldap/servers/plugins/replication/repl5_replica_config.c
+++ b/ldap/servers/plugins/replication/repl5_replica_config.c
@@ -1688,9 +1688,15 @@ replica_cleanallruv_thread(void *arg)
}
/*
* Presetting the rid prevents duplicate thread creation, but allows the db and
changelog to still
- * process updates from the rid. set_cleaned_rid() blocks updates, so we don't
want to do that... yet.
+ * process updates from the rid.
+ * set_cleaned_rid() blocks updates, so we don't want to do that... yet unless
we are in force mode.
+ * If we are forcing a clean independent of state of other servers for this RID we
can set_cleaned_rid()
*/
- preset_cleaned_rid(data->rid);
+ if (data->force) {
+ set_cleaned_rid(data->rid);
+ } else {
+ preset_cleaned_rid(data->rid);
+ }
rid_text = slapi_ch_smprintf("%d", data->rid);
csn_as_string(data->maxcsn, PR_FALSE, csnstr);
/*
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.