<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font face="Times New Roman, Times, serif">This review takes into
account the recommendations of previous review:<br>
</font><br>
<ul>
<li><font face="Times New Roman, Times, serif">Create a Agreement
class in brooker</font></li>
<li><font face="Times New Roman, Times, serif">mv
init/status/schedule/create in that new class</font></li>
<li><font face="Times New Roman, Times, serif">mv
createDefaultReplMgr into the brooker replica class with the
function create_repl_manager()<br>
</font></li>
<li><font face="Times New Roman, Times, serif">Handling of error
condition with exceptions</font></li>
</ul>
<p><font face="Times New Roman, Times, serif">What is not
implemented in that review, that will be implemented with
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/389/ticket/47600">https://fedorahosted.org/389/ticket/47600</a>:<br>
</font></p>
<ul>
<li><font face="Times New Roman, Times, serif">Cleanup of </font>createAgreement
(use of exception). This function will likely be replace by
agreement.create().</li>
<li>Cleanup of enableReplication (use of exception, move under
replica.create()) <br>
</li>
</ul>
<font face="Times New Roman, Times, serif"><a class="moz-txt-link-freetext" href="https://fedorahosted.org/389/attachment/ticket/47590/0003-Ticket-47590-CI-tests-add-split-functions-around-rep.patch">https://fedorahosted.org/389/attachment/ticket/47590/0003-Ticket-47590-CI-tests-add-split-functions-around-rep.patch</a></font>
</body>
</html>