<p>Is this patch useful in existing code base? If the function doesn't exist in the db connector module, it should be not used by the modules, but do an alternative solution. Can you give some example where this is useful now?</p>
<p>Anyhow, as I could understand from the patch, it doesn't do an update on an insert conflict. This can mislead, by seeing the function exported by postgres module, but not fully implemented.</p>
<p>Regarding the commit message, it must be strictly related to what the patch adds/changes to Kamailio sources, not the story of doing the patch. Such details can be in the pull requests description or comments of PR.</p>

<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/1039#issuecomment-287618378">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZdtCxJnp_K8JuuAWtMPe0uon7N7Xks5rnTQcgaJpZM4Mhjex">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZTyvhq1tBSPjlTMip-fC9a-I1gZ1ks5rnTQcgaJpZM4Mhjex.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/1039#issuecomment-287618378"></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":"PERSON","message":"@miconda in #1039: Is this patch useful in existing code base? If the function doesn't exist in the db connector module, it should be not used by the modules, but do an alternative solution. Can you give some example where this is useful now?\r\n\r\nAnyhow, as I could understand from the patch, it doesn't do an update on an insert conflict. This can mislead, by seeing the function exported by postgres module, but not fully implemented.\r\n\r\nRegarding the commit message, it must be strictly related to what the patch adds/changes to Kamailio sources, not the story of doing the patch. Such details can be in the pull requests description or comments of PR."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1039#issuecomment-287618378"}}}</script>