[kamailio/kamailio] db_unixodbc: SQL_NO_DATA should not be considered as an error (#675)
SQL_NO_DATA (100) should not be considered as an error when deletes or updates are done. currently, any return code different than 0 or 1 is considered as an error: a log is generated and the session closed. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/675
-- Commit Summary --
* Merge pull request #1 from kamailio/master * Merge branch 'master' of https://github.com/giovannimele/kamailio * db_unixodbc: do not consider SQL_NO_DATA return code as an error
-- File Changes --
M modules/db_unixodbc/dbase.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/675.patch https://github.com/kamailio/kamailio/pull/675.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/675
Closed #675.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/675#event-696062925
Thanks! Commit pushed manually to avoid the extra two merging notifications from this PR. I think you need to pull and do fast forward before making the pull request not to have those Merge ... commits as part of the PR.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/675#issuecomment-226793001
Hello,
I’ll do it next time.
Thx
Giovanni
From: sr-dev [mailto:sr-dev-bounces@lists.sip-router.org] On Behalf Of Daniel-Constantin Mierla Sent: vendredi 17 juin 2016 17:00 To: Kamailio Devel List Subject: Re: [sr-dev] [kamailio/kamailio] db_unixodbc: SQL_NO_DATA should not be considered as an error (#675)
Thanks! Commit pushed manually to avoid the extra two merging notifications from this PR. I think you need to pull and do fast forward before making the pull request not to have those Merge ... commits as part of the PR.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/kamailio/kamailio/pull/675#issuecomment-226793001, or mute the threadhttps://github.com/notifications/unsubscribe/AF36ZQrl83OQU2flClCvPpSX3PbiG-JGks5qMraIgaJpZM4I4NAU.
-
Daniel-Constantin Mierla -
Giovanni
-
Mele Giovanni