In my test case I was doing an INSERT query... yet db_cassandra would complain there was no result... ( both the log message and return code )
This is the reason I provided the patch. after a little more testing I have found that I get this log message : 0(23827) ERROR: <core> [db_res.c:130]: db_free_result(): invalid parameter So far in my testing everything has performed flawlessly, just with a few less log lines :) in essence this patch simply makes db_cassandra act the same when there is no result set as it does when there are now rows. ( previously it would act like no result set was a big deal ) Jay On 12 March 2014 18:49, Daniel-Constantin Mierla <[email protected]> wrote: > What would be the situation when the query is like SELECT but there is no > result. Is the behaviour as expected with the new patch? > > Anyone here using cassandra having comments? From my point of view is no > problem to push the patch, but I am not using cassandra, so cannot do a > proper review. > > Cheers, > Daniel > > > On 12/03/14 08:53, jay binks wrote: > > If doing a query that returns no results ( Insert etc ) > db_cassa_raw_query would cause these ERRORS to be logged > > 0(22283) ERROR: db_cassandra [dbcassa_base.cpp:739]: > db_cassa_raw_query(): The resultype rows was not set, no point trying to > parse result. > 0(22283) ERROR: avpops [avpops_db.c:333]: db_query_avp(): cannot do the > query > > db_cassa_raw_query would also return -1 as a failure code which caused > avpops_db to log the query failure. > > my patch changes the db_cassa_raw_query log message to debug level, and > returns success from the function. > > I had a quick look to see if there was an elegant way to determine if we > should expect results, so we can vary the response code based on query > type, but I was unable to find anything other than doing string comparisons > on the query, so I opted to not bother with this as it would be erroneous. > > Please find attached patch. > > -- > Sincerely > > Jay > > > _______________________________________________ > sr-dev mailing > [email protected]http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev > > > -- > Daniel-Constantin Mierla - http://www.asipto.comhttp://twitter.com/#!/miconda > - http://www.linkedin.com/in/miconda > Kamailio World Conference - April 2-4, 2014, Berlin, > Germanyhttp://www.kamailioworld.com > > > _______________________________________________ > sr-dev mailing list > [email protected] > http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev > > -- Sincerely Jay
netsip_db_cassandra_rawquery.patch
Description: Binary data
_______________________________________________ sr-dev mailing list [email protected] http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
