Subject RE: [Firebird-Java] Re: Type 2 driver modifications are ready for review.
Author Robert DiFalco
Out of curiosity, what is the value of the Type 2 driver? What are the
main advantages of the Type 4 driver already present with a new Type 2
driver? Are there benchmarks that illustrate the difference in
performance?


-----Original Message-----
From: Ryan Baldwin [mailto:ryan.baldwin@...]
Sent: Wednesday, May 28, 2003 9:03 AM
To: blas@...
Cc: Firebird-Java@yahoogroups.com
Subject: RE: [Firebird-Java] Re: Type 2 driver modifications are ready
for review.


Hi,

I've uploaded firebirdsql.jar to the files area on the yahoo site. Its
name
is firebirdsql_withtype2changes.jar.
This is built from my current working copy using jdk 1.3.1_08. The type
4
functionality (should) be 100% intact.

One thing I am aware of is that TestFBWrappingDataSource::testPooling
always
seems to fail with

testPooling Failure Wrong number of connections! 3, expected 2

junit.framework.AssertionFailedError: Wrong number of connections! 3,
expected 2
at
org.firebirdsql.jdbc.TestFBWrappingDataSource.testPooling(TestFBWrapping
Data
Source.java:178)

This occurs with me when using a fresh checkout also so seems to be
unreleated too my changes.

Please excuse the incomplete post I just made.

Thanks
Ryan


-----Original Message-----
From: Blas Rodriguez Somoza [mailto:blas@...]
Sent: 28 May 2003 14:57
To: Firebird-Java@yahoogroups.com
Subject: Re: [Firebird-Java] Re: Type 2 driver modifications are ready
for review.


Hello

At 27/05/2003 15:27 +0000, Roman Rokytskyy wrote:
>Ryan,
>
> > I've got to the point now where the Type 2 driver modifications made
> > against the head of the CVS tree are ready for review - and in my
> > opinion could be/maybe should be commited before completing the
> > modifications.
> > ...
> > Basicly I would like to have this reviewed and the possibility of
> > having it commited looked at as really I could do with some help in
> > surfacing this through FBDriver and the other methods of obtaining
> > connections which I do not have an extensive understanding of. Also
> > there will be numerous other small things to polish to complete the
> > intergration of these changes - and there are various options for
> > doing these which perahps should be discussed.
> >
> > So if anyone is still interested in having these modifications
> > commited to the tree please let me know and I will make a snapshot
> > of my local CVS tree available through the downloads area and give
> > an thorough overview of what I have had to do and my reasoning.
>
>Personally I would say that you should commit them to the CVS, if your
>changes do not break the driver (at least all test cases should pass).
>I will be able to work on JayBird not earlier than in 2 weeks and my
>plan is to fix multi-threading issues first. Therefore I would like
>that your changes are already in the tree, because it might happen
>that some refactoring is needed.
>
>So, if David and Blas have no objection, I would say "commit".

Only two question for Ryan before:

1.- Can you send me a firebirdsql.jar?. I'll test it.
2.- Why do you want to enable the two types (2 /4) simultaneously and
how
do you intend to achieve that?.

Regards
Blas Rodriguez Somoza




To unsubscribe from this group, send an email to:
Firebird-Java-unsubscribe@yahoogroups.com



Your use of Yahoo! Groups is subject to
http://docs.yahoo.com/info/terms/




Yahoo! Groups Sponsor





To unsubscribe from this group, send an email to:
Firebird-Java-unsubscribe@yahoogroups.com



Your use of Yahoo! Groups is subject to the Yahoo! Terms of Service.