[PATCH] Expose URL property in BaseDataSource

[Loïc PÉRON]

Hi all,

The attached patch does the following:

- changes visibility of Driver.parseUrl to public static
- changes visibility of BaseDataSource.getUrl to public
- add setUrl to BaseDataSourc based on Driver.parseUrl

Adding this property to all datasources simplifies their set-up by
using the same url/user/pass as with DriverManager.

--
Loïc PÉRON
mailto:loic.peron<at>bigfoot.com

[Dave Cramer]

Looking at the spec these are not part of the Datasource API ? Or am I mistaken ?

Dave Cramer

dave.cramer(at)credativ(dot)ca
http://www.credativ.ca

--
Sent via pgsql-jdbc mailing list (pgsql...@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-jdbc

[Loïc PÉRON]

2013/3/11 Dave Cramer <p...@fastcrypt.com>:

> Looking at the spec these are not part of the Datasource API ? Or am I
> mistaken ?

No they are not, but in fact no connection property is mandated by the
spec regarding DataSources.

It is simply a convenient way of configuring the DataSource, similar
to the configuration through DriverManager: url+user+pass

As the URL parsing is already present in Driver, and the URL building
in BaseDataaSource, I just connected url parsing to the BaseDataSource
attributes.

We currently use Oracle, HSQLDB and Postgresql, which is the only one
not exposing a URL property for datasources.

--
Lo�c P�RON
phone:(33) 683 880 177

[Loïc PÉRON]

Hi,

Is the patch attached to my initial message
(http://www.postgresql.org/message-id/CAFKiveQyuA+PQ310pV3tLB5y...@mail.gmail.com)
acceptable? Does it need more work?

--
Loïc PÉRON