<p>Hello,</p>
<p>We realized that redis_execute should return a negative value in case of failure (just like redis_cmd), instead of always returning 1. This introduced a problem with the behavior when calling redis_execute without giving a server name. Since this function will loop through all the servers and execute the pipelined commands for each one, in case of failure the user cannot determine for what server the error occurred, by only returning a negative value. This is why we decided to remove the possibility to call redis_execute with no arguments, and have it loop through all defined servers. The user should only call redis_execute(srv_name) and check the return value for this.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/kamailio/kamailio/pull/1112'>https://github.com/kamailio/kamailio/pull/1112</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>ndb_redis: remove redis_execute command with no arguments</li>
  <li>ndb_redis: make redis_execute() return negative value on error</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/1112/files#diff-0">src/modules/ndb_redis/doc/ndb_redis_admin.xml</a>
    (10)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/1112/files#diff-1">src/modules/ndb_redis/ndb_redis_mod.c</a>
    (22)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/1112/files#diff-2">src/modules/ndb_redis/redis_client.c</a>
    (20)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/1112/files#diff-3">src/modules/ndb_redis/redis_client.h</a>
    (1)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/kamailio/kamailio/pull/1112.patch'>https://github.com/kamailio/kamailio/pull/1112.patch</a></li>
  <li><a href='https://github.com/kamailio/kamailio/pull/1112.diff'>https://github.com/kamailio/kamailio/pull/1112.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/1112">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZePHtU7s5Uwsogoqk1z8BbH2UBWkks5r1yBXgaJpZM4NOEkI">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZSaKhpSVHIK-5TaiJXS1SQ9W88gdks5r1yBXgaJpZM4NOEkI.gif" 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/1112"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Redis remove redis_execute() command with no arguments (#1112)"}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1112"}}}</script>