On Mon, 26 Apr 2004 14:48:45 -0400
Dossy <[EMAIL PROTECTED]> wrote:

> What's funny is if you only implement DbFn_Exec and not DbFn_Select
> and DbFn_DML, the DB API will call the DbFn_Exec when you try [ns_db
> select] and [ns_db dml] -- it's clever that way.

        As I said in my previous post, I don't really see much use for Select
and DML, and I suppose they're in the API for historical reasons. I was
going to suggest that in nsodbc or nsunixodbc these functions should
just call Exec...but it seems that that would be the effect of leaving
them out altogether.  :-)


> Implementing DbFn_DbType is trivial.  :-)

        True enough.


> I'm thinking that given the fixes I've put into the original nsodbc,
> the only things it's currently missing:
>
> 1) More "informative" logging -- not exactly sure what this means.
>    I need to find out.
>
> 2) ODBC v3 support -- yes, I should be able to add this to nsodbc, at
>    worst by merging the changes in nsunixodbc back into nsodbc.
>
> 3) Transaction support.  Trivial change to nsodbc.
>
> > I don't know of a good reason why we couldn't make a single ODBC
> > module that works on all platforms (and maybe clean up the
> > documentation to state so), as well as add some new functionality to
> > it (e.g., explicit transaction support); and I think that Tom's
> > module would be the best base for that.
>
> Why?  nsodbc, as far as my testing went, works ... AND supports
> Redbrick ODBC as well, which I'm sure nsunixodbc probably doesn't.

        Not to minimize the issue of supporting Redbrick, but as far as I can
tell this just consists of an ifdef to include specially named versions
of sql.h and sqlext.h for Redbrick. Trivial change either way.

        I'll have to take a look at your patch for nsodbc and then compare
the two modules to see if I can figure out what advantages nsodbc might
offer. I think you should assess which approach involves less work and
is less error-prone: merging the changes from nsunixodbc into the
existing nsodbc, or merging whatever you feel is needed from nsodbc into
nsunixodbc.


Cheers,
Bob


--
AOLserver - http://www.aolserver.com/

To Remove yourself from this list, simply send an email to <[EMAIL PROTECTED]> with the
body of "SIGNOFF AOLSERVER" in the email message. You can leave the Subject: field of 
your email blank.

Reply via email to