<div dir="ltr"><div>Ok, I need to dig into this anyways so I will see what I can find and work up a patch based on that.<br><br></div><div>Regards,<br><br>Ryan<br></div><div><br></div><div><br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Apr 18, 2013 at 7:41 AM, Vlad Paiu <span dir="ltr">&lt;<a href="mailto:vladpaiu@opensips.org" target="_blank">vladpaiu@opensips.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<u></u>

  
    
    
  
  <div bgcolor="#ffffff" text="#000000">
    Hi Ryan,<br>
    <br>
    Wondered about that myself when the module was built, about why they
    would have two different mechanisms for reporting errors. I included
    both just for the sake of reporting all errors, but didn&#39;t see the
    lcb_get_last_error() triggering.<br>
    I guess we need the help of some libcouchbase devs here, or dig into
    the libcouchbase code to see when lcb_get_last_error() triggers and
    error.<br>
    <br>
    Best Regards,<br>
    <pre cols="72">Vlad Paiu
OpenSIPS Developer
<a href="http://www.opensips-solutions.com" target="_blank">http://www.opensips-solutions.com</a> </pre><div><div class="h5">
    <br>
    On 04/18/2013 12:54 AM, Ryan Bullock wrote:
    </div></div><blockquote type="cite"><div><div class="h5">
      <div dir="ltr">
        <div>
          <div>
            <div>
              <div>Hey all,<br>
                <br>
              </div>
              I am doing some work with opensips + couchbase and have
              started on a patchset for the opensips couchbase driver to
              clean up a few things (better error outputs, etc) and make
              some changes that we require before going production.<br>
              <br>
            </div>
            One thing I am noticing are calls to lcb_get_last_error()
            after every operation, even though the error on the callback
            is being stored in last_error. In my quick testing the
            response from lcb_get_last_error() seems to always be
            success even if the operation failed (such as due to a
            connection failure), last_error shows the correct failure.<br>
            <br>
            I am wondering if the checks to lcb_get_last_error could be
            removed? Or are their instances where it would show an error
            not caught by last_error? I don&#39;t see many uses of
            lcb_get_last_error in the libcouchbase examples and they
            seem to all go off the error in the callback. The benefit I
            see is being able to more easily report useful and relevant
            errors.<br>
            <br>
          </div>
          Input welcome.<br>
          <br>
        </div>
        Regards,<br>
        <br>
        Ryan<br>
        <div>
          <div><br>
            <br>
          </div>
        </div>
      </div>
      </div></div><pre><fieldset></fieldset>
_______________________________________________
Devel mailing list
<a href="mailto:Devel@lists.opensips.org" target="_blank">Devel@lists.opensips.org</a>
<a href="http://lists.opensips.org/cgi-bin/mailman/listinfo/devel" target="_blank">http://lists.opensips.org/cgi-bin/mailman/listinfo/devel</a>
</pre>
    </blockquote>
  </div>

<br>_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.opensips.org">Devel@lists.opensips.org</a><br>
<a href="http://lists.opensips.org/cgi-bin/mailman/listinfo/devel" target="_blank">http://lists.opensips.org/cgi-bin/mailman/listinfo/devel</a><br>
<br></blockquote></div><br></div>