| From: | Craig Ringer <ringerc(at)ringerc(dot)id(dot)au> |
|---|---|
| To: | Mikko Tiihonen <Mikko(dot)Tiihonen(at)nitorcreations(dot)com> |
| Cc: | Kris Jurka <books(at)ejurka(dot)com>, Sérgio Saquetim <sergiosaquetim(at)gmail(dot)com>, "pgsql-jdbc(at)postgresql(dot)org" <pgsql-jdbc(at)postgresql(dot)org>, Rafael dos Santos Silva <xfalcox(at)gmail(dot)com> |
| Subject: | Re: Severe performance degradation when using the 9.2-1000 JDBC 4 driver |
| Date: | 2012-10-19 07:22:17 |
| Message-ID: | 5080FFA9.70402@ringerc.id.au |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-jdbc |
On 10/19/2012 02:36 PM, Mikko Tiihonen wrote:
>>> I think the DNS lookup is part of the JDBC failover connection patch
>>> that I created. I initially used a InetSocketAddress.getHostString()
>>> that does not do any DNS lookups, but since that method was added in
>>> Java7 I had to revert to getHostName() method which does.
>>>
>>
>> Is there any functionality that we need that is provided by using
>> InetSocketAddress or is it just a convenient object to pass around host
>> and port together? The simplest option to me seems to be reverting to
>> passing simple strings/ints around (possibly in some kind of our own
>> container).
>
> It is just a convenient container. And it seems on older JVMs it cannot be used for such purposes.
> I agree that we should just replace it with our own container that contains the host/port pair.
If that's all you're using it for, then that's the simplest and best
choice. +1 from me on that.
--
Craig Ringer
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Scott Harrington | 2012-10-19 13:24:49 | Re: Severe performance degradation when using the 9.2-1000 JDBC 4 driver |
| Previous Message | Mikko Tiihonen | 2012-10-19 06:36:11 | Re: Severe performance degradation when using the 9.2-1000 JDBC 4 driver |