On 1/24/06, Rob Kinyon <[EMAIL PROTECTED]> wrote:
> The reason I wrote the patch is so that I would use indices for
> numeric columns. If MySQL has to convert the value from a string to a
> number, it doesn't use an index. You might want to consider that when
> dealing with stuff.

That shouldn't be an issue unless someone passes a value like " 1 " to
an integer attribute or something, right?

> Also, you might want to consider using bind_param() instead of passing
> parameters to execute() directly. This patch only deals with
> parameters that don't have a type associated with them. If you use
> bind_param() instead, then you can associate the type and DBD::mysql
> shouldn't auto-quote it.

I know about bind_param(), I'm just trying to avoid having to call it.
 My impression is that iterating over all bind values and calling
bind_param() repeatedly is going to be slower than calling execute()
with args.  Maybe I'll be forced to do it anyway and then we'll see...

-John


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid3432&bid#0486&dat1642
_______________________________________________
Rose-db-object mailing list
Rose-db-object@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rose-db-object

Reply via email to