So I've been playing with Sequel and really like it.  Despite some
conflicting info on the web, though, it appears the Oracle driver does
NOT currently have support for bind variables.  I'm basing that on
this:

http://groups.google.com/group/sequel-talk/browse_thread/thread/6451d8d417fdbcc/ce379e95207707d2?lnk=gst&q=bind+variables#ce379e95207707d2
http://groups.google.com/group/sequel-talk/browse_thread/thread/c58116631010b400/7c78f63505028582?lnk=gst&q=bind+variables#7c78f63505028582

As well as comparing these:

http://github.com/jeremyevans/sequel/blob/master/lib/sequel/adapters/postgres.rb
http://github.com/jeremyevans/sequel/blob/master/lib/sequel/adapters/oracle.rb

Specifically it seems the issue is:

"The Sequel::Oracle::Database#execute statement takes a string.
Modifying it to take a string and an array of bind variables is
possible, but making sure that Sequel::Oracle::Dataset calls execute
with bind variables instead of a single string may be quite a bit of
work"

>From an initial reading of the Sequel codebase, it appears there is
now support to pass bind variables around, since postgres.rb appears
to do this in most (all?) cases.

So my actual question is:  If I cut-and-paste the postgres.rb
functions that do stuff with bind variables, should this be sufficient
to add bind support in all cases?  Or are there other pieces that need
to be modified in the higher-level sequel.rb functions?  I don't quite
grok sql.rb yet, so I'm not sure if there are any lingering hardcoded
sql strings that need to become @sql, @args.

BTW - really like the codebase so far.  I tried adding bind variable
support to AR and felt like I wanted to stab myself in the face.

Thanks,
Nate

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sequel-talk?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to