<p>I start reviewing the patches and : </p>

<ul class="task-list">
<li>the first patch - <a href="https://github.com/OpenSIPS/opensips/commit/a7549e00bdd3fd13fc20777bccf935d41303980e" class="commit-link"><tt>a7549e0</tt></a> - I do not agree with the approach - once you started a LB session, you should not change the set or resources.  If you need to do that, first reset the existing LB session and then create a new one.</li>
<li>patch 2 and 3 are ok as functionality - please make a separate PR with them, so I can accept them.</li>
<li>patch 4 - I do not see any reason for defining the _NEG algs, as the only difference is about handling the negative values - we could simply do that all the time....</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/OpenSIPS/opensips/pull/345#issuecomment-59343939">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/5479561__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyOTA3NTU2NywiZGF0YSI6eyJpZCI6NDQ0NDI0ODV9fQ==--f78db68921bc21af812552a41eab6f41139d7200.gif" width="1" /></p>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"View this Pull Request on GitHub","action":{"@type":"ViewAction","url":"https://github.com/OpenSIPS/opensips/pull/345#issuecomment-59343939","name":"View Pull Request"}}</script>