<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
I pushed a rework-ed patch. Can you try with the patch from commit:<br>
<br>
-
<a class="moz-txt-link-freetext" href="http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3cde37cf10795b4eb785ffd141a8b28e64864f5d">http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3cde37cf10795b4eb785ffd141a8b28e64864f5d</a><br>
<br>
I don't use cassandra, so didn't test it. If all goes fine with the
tests, then I can backport.<br>
<br>
Cheersm<br>
Daniel<br>
<br>
<br>
<div class="moz-cite-prefix">On 06/02/14 05:30, jay binks wrote:<br>
</div>
<blockquote
cite="mid:CAO47HpcRVyCZJ7T1oZeUtL2Fs81tw0m0uOR_b-Ye+KFAidz3bA@mail.gmail.com"
type="cite">
<div dir="ltr">So I posted this to the user list in December, and
got nowhere..
<div><br>
</div>
<div>Can someone familiar with db_cassandra review my patch (
possibly clean this up and commit it )</div>
<div>
<div class="gmail_quote">
<div dir="ltr">
<div><br>
</div>
<div><br>
</div>
<div>
<div># git diff dbcassa_base.cpp </div>
<div>diff --git a/modules/db_cassandra/dbcassa_base.cpp
b/modules/db_cassandra/dbcassa_base.cpp</div>
<div>index e9d3a32..155221d 100644</div>
<div>--- a/modules/db_cassandra/dbcassa_base.cpp</div>
<div>+++ b/modules/db_cassandra/dbcassa_base.cpp</div>
<div>@@ -439,6 +439,7 @@ ColumnVecPtr
cassa_translate_query(const db1_con_t* _h, const
db_key_t* _k,</div>
<div> int key_len=0, seckey_len = 0;</div>
<div> int no_kc, no_sec_kc;</div>
<div> dbcassa_table_p tbc;</div>
<div>+ char pk[255];</div>
<div> </div>
<div> /** Lock table schema and construct primary
and secondary key **/</div>
<div> if(_k) {</div>
<div>@@ -495,8 +496,12 @@ ColumnVecPtr
cassa_translate_query(const db1_con_t* _h, const
db_key_t* _k,</div>
<div> } else { /* the table doesn't have
any secondary key defined */</div>
<div> if(_c) {</div>
<div class="im">
<div> for(int i=0;
i< _nc; i++) {</div>
</div>
<div>-
sp.column_names.push_back(_c[i]->s);</div>
<div class="im">
<div>-
LM_DBG("Query col: %s\n", _c[i]->s);</div>
</div>
<div>+ sprintf(pk,
"%.*s", _c[i]->len, _c[i]->s );</div>
<div>
+
sp.column_names.push_back( pk);</div>
<div>+
//sp.column_names.push_back(_c[i]->s);</div>
<div>+
LM_DBG("Query col: %s\n", pk );</div>
<div>+
//LM_DBG("Query col: %s\n", _c[i]->s);</div>
<div class="im">
<div>+
LM_DBG("JAY Query col: %.*s\n", _c[i]->len,
_c[i]->s);</div>
<div> }</div>
</div>
<div class="im">
<div> LM_DBG("get %d
columns\n", _nc);</div>
</div>
<div class="im">
<div>
sp.__isset.column_names = true; // set</div>
</div>
</div>
<div><br>
</div>
<div><br>
</div>
<div><br>
</div>
<div><br>
</div>
<div>yea I know I left crap in there, but it gives you an
idea... also...</div>
<div>yea pk is a crap name... copy paste... and I havnt
given any thought to the size 255 , but thats probably
fairly safe and anything smaller might not be so safe.</div>
<div>( unless there is a limit elsewhere I should observe
)</div>
<div><br>
</div>
<div>Jay</div>
</div>
</div>
</div>
</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>
<pre class="moz-signature" cols="72">--
Daniel-Constantin Mierla - <a class="moz-txt-link-freetext" href="http://www.asipto.com">http://www.asipto.com</a>
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda">http://www.linkedin.com/in/miconda</a></pre>
</body>
</html>