Thanks for the note.  Sounds like a good plan.  The patch is handy for my
work in the meantime, regardless.

Question -- will there be a default adapter, suitable for use when the user
doesn't specify one?  That would remove some confusing error messages.

Incidentally, on a similar note, it would be nice if a useful error message
is given when the user name, password, or database URL are incorrect.
Currently a null exception is thrown instead.

Best, WILL

-----Original Message-----
From: John McNally [mailto:[EMAIL PROTECTED]]
Sent: Monday, May 13, 2002 1:43 PM
To: Turbine Torque Developers List
Subject: Re: patches -- backlog?


On the patch to add back the adapter property, I have some code in the
works to remove torque's embedded connection pool and use standard jdbc2
DataSources in its place.  I thought I would have that done by today,
but it touched a bit more code than I originally thought.  When I check
in this change, the way torque's pool is configured will be separated
from the choice of adapter.  The adapter will be set as a property
similar to your proposal, but it will have no relationship to drivers.
Not sure I will have time to get this in this week, but if not I will
make another effort next weekend.

john mcnally

On Mon, 2002-05-13 at 12:58, Will Glass-Husain wrote:
> Hi,
>
> I submitted a couple of patches about 10 days ago to this list.  One of
them
> is in response to a question I've seen on the list a few times (and is in
a
> Buzilla note as well).  Specifically, it is a patch to DBFactory that
> reinstates the "adapter" property, allowing users to use any database
driver
> with Torque.  (Currently, there is a list of drivers hard coded, but other
> drivers are not allowed).   This is particularly important patch for those
> using the Sun ODBC-JDBC driver, which may be used with multiple databases.
>
> The other patches are various changes to the SQL templates.  For Microsoft
> SQL, they fix an important bug.  (currently, the create table SQL for MS
SQL
> doesn't work with multiple tables or with date fields).
>
> Just curious where those patches stands.  Is there a backlog of things to
> do, and the committers just haven't caught up, or is there a problem with
> the patches themselves?
>
> The patches are at:
>
>
http://www.mail-archive.com/[email protected]/msg00219.h
> tml
>
http://www.mail-archive.com/[email protected]/msg00220.h
> tml
>
http://www.mail-archive.com/[email protected]/msg00221.h
> tml
>
> Let me know if I can help with anything.
>
> Best, WILL
>
>
> _______________________________________
> Forio Business Simulations
> Will Glass-Husain
> (415) 440-7500 phone
> (415) 235-4293 mobile
>
> [EMAIL PROTECTED]
> www.forio.com
>
>



--
To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>



--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to