[Mono-dev] Improving memory usage in /System.Web.SessionState/SessionSQLServerHandler.cs

[Hubert FONGARNAND]

Could someone commit this patch?
Thanks

Le lundi 27 novembre 2006 à 12:17 +0100, Hubert FONGARNAND a écrit :

> Hi,
> 
> When using Npgsql, if you try to send (INSERT or UPDATE) a bytea
> (blob) element without using "Prepare" it will be sent as a Escaped
> string... This string is 4x bigger than the real bytea...
> So : The memory usage on the ASP.NET server is enormous when the blob
> is more than 1MB... and the network traffic is enormous too!
> (see
> http://pgfoundry.org/tracker/index.php?func=detail&aid=1000458&group_id=1000140&atid=590)
> 
> If you want to avoid that you've to use Prepare before executing
> ExecuteNonQuery, or ExecuteReader...
> If you don't believe me use Ethereal (wireshark)
> 
> Here's a patch that add a cmd.Prepare allowing Npgsql sending data in
> binary format!
> This patch shouldn't change anything if you use other connector than
> Npgsql.
> 
> Thanks to review it!
> 
> Hubert FONGARNAND
> 
> 
> 
> 
> Index: mcs/class/System.Web/System.Web.SessionState/SessionSQLServerHandler.cs
> ===================================================================
> --- mcs/class/System.Web/System.Web.SessionState/SessionSQLServerHandler.cs     (révision 68495)
> +++ mcs/class/System.Web/System.Web.SessionState/SessionSQLServerHandler.cs     (copie de travail)
> @@ -196,6 +196,7 @@
>                         command.CommandText = selectCommand;
>                         command.Parameters.Add (CreateParam (command, DbType.String, "SessionID", id));
>                         command.Parameters.Add (CreateParam (command, DbType.DateTime, "Expires", DateTime.Now ));
> +                       command.Prepare();
>                         return command.ExecuteReader ();
>                 }
>  
> @@ -253,7 +254,7 @@
>                                                    session.StaticObjects.ToByteArray ()));
>                         param.Add (CreateParam (command, DbType.Binary, "SessionData",
>                                                    session.SessionDictionary.ToByteArray ()));
> -
> +                       command.Prepare();
>                         command.ExecuteNonQuery ();
>                 }
>  
> @@ -287,6 +288,7 @@
>                         param.Add (CreateParam (command, DbType.Int32, "Timeout", timeout));
>                         param.Add (CreateParam (command, DbType.Binary, "SessionData",
>                                                                 dict.ToByteArray ()));
> +                       command.Prepare();
>  
>                         command.ExecuteNonQuery ();
>                 }
> 
> 
> 
> _______________________________________________
> Ce message et les ventuels documents joints peuvent contenir des
> informations confidentielles.
> Au cas oäµ² il ne vous serait pas destin, nous vous remercions de bien
> vouloir le supprimer et en aviser immæ´¸diatement l'expditeur. Toute
> utilisation de ce message non conforme ⭁ sa destination, toute
> diffusion ou publication, totale ou partielle et quel qu'en soit le
> moyen est formellement interdite.
> Les communications sur internet n'tant pas s牌curises, l'intߜgrit de
> ce message n'est pas assurŨe et la socit㵵 mettrice ne peut 攮tre
> tenue pour responsable de son contenu. 
> 
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
_______________________________________________
Ce message et les éventuels documents joints peuvent contenir des informations confidentielles.
Au cas où il ne vous serait pas destiné, nous vous remercions de bien vouloir le supprimer et en aviser immédiatement l'expéditeur. Toute utilisation de ce message non conforme à sa destination, toute diffusion ou publication, totale ou partielle et quel qu'en soit le moyen est formellement interdite.
Les communications sur internet n'étant pas sécurisées, l'intégrité de ce message n'est pas assurée et la société émettrice ne peut être tenue pour responsable de son contenu.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/mono-devel-list/attachments/20061128/ea926d90/attachment.html