[asterisk-commits] tilghman: trunk r117264 - in /trunk: CHANGES res/res_odbc.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Tue May 20 11:25:16 2008
New Revision: 117264

URL: http://svn.digium.com/view/asterisk?view=rev&rev=117264
Log:
Increase limit of unshared connections from 1023 to 4.2 billion.
(Related to issue #12677)

Modified:
    trunk/CHANGES
    trunk/res/res_odbc.c

Modified: trunk/CHANGES
URL: http://svn.digium.com/view/asterisk/trunk/CHANGES?view=diff&rev=117264&r1=117263&r2=117264
==============================================================================
--- trunk/CHANGES (original)
+++ trunk/CHANGES Tue May 20 11:25:16 2008
@@ -14,6 +14,13 @@
    based on other parameters.  The default is still to search based on the
    forwarding station ID.  However, there are new options that allow you to search
    based on the message desk terminal ID, or the message desk number.
+ * TIMEOUT() has been modified to be accurate down to the millisecond.
+ * ENUM*() functions now include the following new options:
+     - 'u' returns the full URI and does not strip off the URI-scheme.
+	 - 's' triggers ISN specific rewriting
+	 - 'i' looks for branches into an Infrastructure ENUM tree
+	 - 'd' for a direct DNS lookup without any flipping of digits.
+ * TXCIDNAME() has a new zone-suffix parameter (which defaults to 'e164.arpa')
 
 Zaptel channel driver (chan_zap) Changes
 ----------------------------------------
@@ -106,20 +113,16 @@
   * Addresses managed by DNS manager now can check to see if there is a DNS
     SRV record for a given domain and will use that hostname/port if present.
 
-Dialplan function changes
--------------------------
- * TIMEOUT() has been modified to be accurate down to the millisecond.
- * ENUM*() functions now include the following new options:
-     - 'u' returns the full URI and does not strip off the URI-scheme.
-	 - 's' triggers ISN specific rewriting
-	 - 'i' looks for branches into an Infrastructure ENUM tree
-	 - 'd' for a direct DNS lookup without any flipping of digits.
- * TXCIDNAME() has a new zone-suffix parameter (which defaults to 'e164.arpa')
-
 AMI - The manager (TCP/TLS/HTTP)
 --------------------------------
   * The Status command now takes an optional list of variables to display
     along with channel status.
+
+ODBC Changes
+------------
+  * res_odbc no longer has a limit of 1023 total possible unshared connections,
+    as some people were running into this limit.  This limit has been increased
+    to 4.2 billion.
 
 ------------------------------------------------------------------------------
 --- Functionality changes from Asterisk 1.4.X to Asterisk 1.6.0  -------------

Modified: trunk/res/res_odbc.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_odbc.c?view=diff&rev=117264&r1=117263&r2=117264
==============================================================================
--- trunk/res/res_odbc.c (original)
+++ trunk/res/res_odbc.c Tue May 20 11:25:16 2008
@@ -59,10 +59,10 @@
 	char *sanitysql;
 	SQLHENV env;
 	unsigned int haspool:1;              /* Boolean - TDS databases need this */
-	unsigned int limit:10;               /* Gives a limit of 1023 maximum */
-	unsigned int count:10;               /* Running count of pooled connections */
 	unsigned int delme:1;                /* Purge the class */
 	unsigned int backslash_is_escape:1;  /* On this database, the backslash is a native escape sequence */
+	unsigned int limit;                  /* 1023 wasn't enough for some people */
+	unsigned int count;                  /* Running count of pooled connections */
 	unsigned int idlecheck;              /* Recheck the connection if it is idle for this long */
 	struct ao2_container *obj_container;
 };