<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hello,<br>
      <br>
      I do not like the idea behind the patch. If redis database is
      down, what is the point of launching kamailio anyway?<br>
      If you need redis to run you should fix it, or not to use the
      ndb_redis module if you want to run without redis.<br>
      <br>
      Any other opinions are welcome.<br>
      <br>
      Cheers,<br>
      Vicente.<br>
      <br>
      <br>
      <br>
      On 04/13/2015 04:24 PM, Giacomo Vacca wrote:<br>
    </div>
    <blockquote cite="mid:kamailio%2Fkamailio%2Fpull%2F121@github.com"
      type="cite">
      <hr>
      <h4>You can view, comment on, or merge this pull request online
        at:</h4>
      <p>  <a moz-do-not-send="true"
          href="https://github.com/kamailio/kamailio/pull/121">https://github.com/kamailio/kamailio/pull/121</a></p>
      <h4>Commit Summary</h4>
      <ul>
        <li>ndb_redis: optionally allow mod init with redis unavailable</li>
        <li>ndb_redis: update doc with init_without_redis</li>
        <li>ndb_redis: update README with init_without_redis</li>
      </ul>
      <h4>File Changes</h4>
      <ul>
        <li> <strong>M</strong> <a moz-do-not-send="true"
            href="https://github.com/kamailio/kamailio/pull/121/files#diff-0">modules/ndb_redis/README</a>
          (28) </li>
        <li> <strong>M</strong> <a moz-do-not-send="true"
            href="https://github.com/kamailio/kamailio/pull/121/files#diff-1">modules/ndb_redis/doc/ndb_redis_admin.xml</a>
          (19) </li>
        <li> <strong>M</strong> <a moz-do-not-send="true"
            href="https://github.com/kamailio/kamailio/pull/121/files#diff-2">modules/ndb_redis/ndb_redis_mod.c</a>
          (13) </li>
      </ul>
      <h4>Patch Links:</h4>
      <ul>
        <li><a moz-do-not-send="true"
            href="https://github.com/kamailio/kamailio/pull/121.patch">https://github.com/kamailio/kamailio/pull/121.patch</a></li>
        <li><a moz-do-not-send="true"
            href="https://github.com/kamailio/kamailio/pull/121.diff">https://github.com/kamailio/kamailio/pull/121.diff</a></li>
      </ul>
      <p
        style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>
        Reply to this email directly or <a moz-do-not-send="true"
          href="https://github.com/kamailio/kamailio/pull/121">view it
          on GitHub</a>.<img moz-do-not-send="true" alt=""
src="https://github.com/notifications/beacon/AF36ZZop9fhtMFD5OO34LcPnMaG9CuPkks5n-8kQgaJpZM4D_ciZ.gif"
          height="1" width="1"></p>
      <div itemscope="" itemtype="http://schema.org/EmailMessage">
        <div itemprop="action" itemscope=""
          itemtype="http://schema.org/ViewAction">
          <link itemprop="url"
            href="https://github.com/kamailio/kamailio/pull/121">
          <meta itemprop="name" content="View Pull Request">
        </div>
        <meta itemprop="description" content="View this Pull Request on
          GitHub">
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
sr-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a>
<a class="moz-txt-link-freetext" href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>