Re: Adding new dependencies for in-core

From: Dave Cramer <pg(at)fastcrypt(dot)com>
To: Vladimir Sitnikov <sitnikov(dot)vladimir(at)gmail(dot)com>
Cc: List <pgsql-jdbc(at)postgresql(dot)org>
Subject: Re: Adding new dependencies for in-core
Date: 2015-06-30 12:32:34
Message-ID: CADK3HHKXFh1w9YJT4KHxLrabgvjmVB48C8F-4Yb=8ACtVddu5A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

Hi Vladimir,

There are a large number of people still downloading jars, so simply using
maven dependency doesn't work.

It would have to be shaded IMO.

This is why I have avoided dependencies in the past.

Dave Cramer

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

On 30 June 2015 at 08:21, Vladimir Sitnikov <sitnikov(dot)vladimir(at)gmail(dot)com>
wrote:

> Hi,
>
> I've had a quick look into https://github.com/pgjdbc/pgjdbc/issues/345
> (Share parsed query texts across different connections) and it looks
> like https://github.com/ben-manes/concurrentlinkedhashmap would be
> useful to implement the cache.
>
> We don't need all the stuff CLHM has, but I see no easy&scalable way
> of doing concurrent map with eviction by mere java.util.concurrent
> stuff.
>
> Well, I can do lock splitting (i.e. use multiple small LRU caches
> instead of a single big cache), however that looks like reinventing
> the wheel.
>
> Can you suggest what is the best way of adding CLHM dependency?
> Should it be added in plain?
> Should it be shaded? (i.e. renamed to org.postgresql.clhm...)
>
> I'm more inclined to the "add regular dependency" approach.
>
> I expect similar question might appear if we consider using netty for IO.
>
> --
> Regards,
> Vladimir Sitnikov
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>

In response to

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Vladimir Sitnikov 2015-06-30 15:47:38 Re: Adding new dependencies for in-core
Previous Message Vladimir Sitnikov 2015-06-30 12:21:19 Adding new dependencies for in-core