<HTML>
<HEAD>
<TITLE>Re: [Freeswitch-users] Suggestion to harmonize 'hash' &amp; 'db' insert method</TITLE>
</HEAD>
<BODY>
<FONT FACE="Monaco, Courier New"><SPAN STYLE='font-size:11pt'>You have to keep in mind that we have to support more then just mysql or sqlite with the the db interface... This means using SQL that&#8217;s common to ALL platforms include MSSQL, PostgreSQL, Oracle and any other database someone might want to use via ODBC... <BR>
<BR>
So to do this completely would require modification to the code on insert to first check if the row exists for the key, then deleting it, then inserting the new row... &nbsp;This is going to be the most generic supportable method to make them work consistently across the board... <BR>
<BR>
However, Keep in mind that what you are calling a bug, could be treated by many people as a feature... Don&#8217;t allow me to create/replace and already existing key...<BR>
<BR>
<BR>
<BR>
<BR>
On 11/18/12 7:26 AM, &quot;Scott&quot; &lt;<a href="8f27e956@gmail.com">8f27e956@gmail.com</a>&gt; wrote:<BR>
<BR>
</SPAN></FONT><BLOCKQUOTE><FONT FACE="Monaco, Courier New"><SPAN STYLE='font-size:11pt'>OK, with the one-time </SPAN></FONT><SPAN STYLE='font-size:11pt'><FONT FACE="Courier New">CREATE UNIQUE INDEX on 'table' (hostname,realm,data_key); </FONT><FONT FACE="Monaco, Courier New">stmt in place at the schema-level, a portable work-a-like of INSERT OR REPLACE stmt is the UPDATE-INSERT (i.e. &quot;UPSERT&quot; lol) stmt pair that follows,<BR>
<BR>
</FONT><FONT FACE="Courier New">BEGIN TRANSACTION;                                                                                                                 <BR>
UPDATE 'table' SET data_value='value_data' WHERE hostname='value_hostname' AND realm='value_realm' AND data_key='value_data_key' ;<BR>
INSERT INTO 'table' (hostname,realm,data_key,data_value) SELECT 'value_hostname', 'value_realm', 'value_data_key', 'value_data_value'<BR>
   WHERE NOT EXISTS (SELECT 1 FROM 'table' WHERE hostname='value_hostname' AND realm='value_realm' AND data_key='value_data_key' );<BR>
COMMIT;                                                                                                                             <BR>
</FONT><FONT FACE="Monaco, Courier New"><BR>
The two WHERE clauses must be identical and the variables should be in the same order as the unique compound index is ordered.<BR>
The UPDATE fails quietly and benignly if the record does NOT already exist and succeeds if it does.<BR>
The INSERT ... WHERE NOT EXISTS stmt succeeds if the record does NOT already exist fails quietly and benignly if it does. <BR>
The BEGIN and COMMIT aren't typically necessary but they're over-safe just in case the same record is attempted by another thread.<BR>
<BR>
I think this get's it done.<BR>
<BR>
:-)<BR>
<BR>
<BR>
On 18 November 2012 06:25, S. Scott &lt;<a href="8f27e956@gmail.com">8f27e956@gmail.com</a>&gt; wrote:<BR>
</FONT></SPAN><BLOCKQUOTE><SPAN STYLE='font-size:11pt'><FONT FACE="Monaco, Courier New">The IF NOT EXITS term is easily omitted.  It's a helper term for batch scripted SQL to prevent an error being thrown in the case of the index already exists as in the case where the make schema script is run more than once.  Easily omitted and, perhaps more properly, replaced with a DROP INDEX stmt and CREATE UNIQUE INDEX stmt pair.<BR>
<BR>
Yeah, i checked further, the OR REPLACE modifier is a little more Hit/miss.  The capability is there typically present, but the syntax varies -- e.g sqlite3 ON REPLACE is mySQL's ON DUPLICATE KEY UPDATE.<BR>
<BR>
Darn shame sqlite3 doesn't support stored procedures 'cause then it's more easily abstracted for portability up the SQL server engine curve.<BR>
<BR>
On 2012-11-18, at 3:53, Gerald Weber &lt;<a href="gerald.weber@besharp.at">gerald.weber@besharp.at</a>&gt; wrote:<BR>
<BR>
</FONT></SPAN><BLOCKQUOTE><SPAN STYLE='font-size:11pt'><FONT FACE="Monaco, Courier New">AFAIK, &#8222;if not exists&#8220; and &#8222;or replace&#8220; is not ansi sql  syntax, so you will get in trouble if <BR>
someone uses odbc to a db that doesn&#8217;t support non standard syntax.<BR>
 <BR>
The insert or replace could be replaced with the merge statement.<BR>
 <BR>
</FONT></SPAN><FONT FACE="Monaco, Courier New"><FONT SIZE="2"><SPAN STYLE='font-size:10pt'><B>Von:</B> <a href="freeswitch-users-bounces@lists.freeswitch.org">freeswitch-users-bounces@lists.freeswitch.org</a> [<a href="mailto:freeswitch-users-bounces@lists.freeswitch.org">mailto:freeswitch-users-bounces@lists.freeswitch.org</a>] <B>Im Auftrag von </B>Scott<BR>
<B>Gesendet:</B> Sonntag, 18. November 2012 02:42<BR>
<B>An:</B> FreeSWITCH Users Help<BR>
<B>Betreff:</B> [Freeswitch-users] Suggestion to harmonize 'hash' &amp; 'db' insert method<BR>
</SPAN></FONT><SPAN STYLE='font-size:11pt'> <BR>
Are there reasons why the function 'hash' (non-persistent storage) and function 'db' (persistent storage) share a look-a-like user interface (e.g. select/insert/delete) but do not work-a-like.  In particular, in 'hash', insert overwrites an identical realm/data_key pair whereas 'db' , in so far as I can tell, just blindly adds, and adds, adds, the insert(s).  However, the 'db' select method returns one record only even if the db has accumulated many realm/data_key records, including data_value duplicates.  <BR>
<BR>
NOTWITHSTANDING reasons-unknow-to-me, a remedy to harmonize the non-persistent 'hash' with the persistent 'db'  I *think* is straight forward ...<BR>
<BR>
(1) At the call_limit.db schema-level, a one-time create UNIQUE COMPOUND index, as follows,<BR>
<BR>
CREATE UNIQUE INDEX IF NOT EXISTS 'idx_db_data_HostRealmDK' ON 'db_data' ('hostname','realm','data_key');<BR>
<BR>
(1) At the c-language embedded SQL string-level,<BR>
<BR>
change the existing INSERT to be the following,<BR>
<BR>
INSERT OR REPLACE INTO db_data (hostname,realm,data_key,data) VALUES (%s,%s,%s,%s);<BR>
<BR>
The index will help speed UP the reads.  Neither change is &quot;exotic&quot; and should be portable across standard sql implementations.<BR>
<BR>
;-) ... thoughts ?<BR>
<BR>
Thanks,<BR>
<BR>
<BR>
_________________________________________________________________________<BR>
Professional FreeSWITCH Consulting Services:<BR>
<a href="consulting@freeswitch.org">consulting@freeswitch.org</a><BR>
<a href="http://www.freeswitchsolutions.com">http://www.freeswitchsolutions.com</a><BR>
<BR>
FreeSWITCH-powered IP PBX: The CudaTel Communication Server<BR>
<a href="http://www.cudatel.com">http://www.cudatel.com</a><BR>
<BR>
Official FreeSWITCH Sites<BR>
<a href="http://www.freeswitch.org">http://www.freeswitch.org</a><BR>
<a href="http://wiki.freeswitch.org">http://wiki.freeswitch.org</a><BR>
<a href="http://www.cluecon.com">http://www.cluecon.com</a><BR>
<BR>
FreeSWITCH-users mailing list<BR>
<a href="FreeSWITCH-users@lists.freeswitch.org">FreeSWITCH-users@lists.freeswitch.org</a><BR>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-users">http://lists.freeswitch.org/mailman/listinfo/freeswitch-users</a><BR>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-users">http://lists.freeswitch.org/mailman/options/freeswitch-users</a><BR>
<a href="http://www.freeswitch.org">http://www.freeswitch.org</a><BR>
</SPAN></FONT></BLOCKQUOTE></BLOCKQUOTE><FONT FACE="Monaco, Courier New"><SPAN STYLE='font-size:11pt'><BR>
<BR>
<HR ALIGN=CENTER SIZE="3" WIDTH="95%"></SPAN></FONT><FONT SIZE="2"><FONT FACE="Consolas, Courier New, Courier"><SPAN STYLE='font-size:10pt'>_________________________________________________________________________<BR>
Professional FreeSWITCH Consulting Services:<BR>
<a href="consulting@freeswitch.org">consulting@freeswitch.org</a><BR>
<a href="http://www.freeswitchsolutions.com">http://www.freeswitchsolutions.com</a><BR>
<BR>
FreeSWITCH-powered IP PBX: The CudaTel Communication Server<BR>
<a href="http://www.cudatel.com">http://www.cudatel.com</a><BR>
<BR>
Official FreeSWITCH Sites<BR>
<a href="http://www.freeswitch.org">http://www.freeswitch.org</a><BR>
<a href="http://wiki.freeswitch.org">http://wiki.freeswitch.org</a><BR>
<a href="http://www.cluecon.com">http://www.cluecon.com</a><BR>
<BR>
FreeSWITCH-users mailing list<BR>
<a href="FreeSWITCH-users@lists.freeswitch.org">FreeSWITCH-users@lists.freeswitch.org</a><BR>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-users">http://lists.freeswitch.org/mailman/listinfo/freeswitch-users</a><BR>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-users">http://lists.freeswitch.org/mailman/options/freeswitch-users</a><BR>
<a href="http://www.freeswitch.org">http://www.freeswitch.org</a><BR>
</SPAN></FONT></FONT></BLOCKQUOTE><FONT SIZE="2"><FONT FACE="Consolas, Courier New, Courier"><SPAN STYLE='font-size:10pt'><BR>
</SPAN></FONT></FONT><FONT FACE="Monaco, Courier New"><SPAN STYLE='font-size:11pt'>-- <BR>
Ken<BR>
<FONT COLOR="#0000FF"><U><a href="http://www.FreeSWITCH.org">http://www.FreeSWITCH.org</a><BR>
<a href="http://www.ClueCon.com">http://www.ClueCon.com</a><BR>
<a href="http://www.OSTAG.org">http://www.OSTAG.org</a><BR>
</U></FONT>irc.freenode.net #freeswitch<BR>
</SPAN></FONT>
</BODY>
</HTML>