I'm OK with requiring Java 8. I don't think we should make Guava a required dependency.
On 12 August 2016 at 21:06, Philippe Mouawad <[email protected]> wrote: > Hi, > Sebb, Milamber, Antonio, > any thoughts on this ? > > > Thanks > > On Thu, Aug 11, 2016 at 7:38 AM, Felix Schumacher < > [email protected]> wrote: > >> >> >> Am 9. August 2016 07:42:39 MESZ, schrieb Philippe Mouawad < >> [email protected]>: >> >Hello, >> >Any thoughts on this ? >> >> To widen the discussion :) >> >> What about adding a full blown cache like commons jcs (is it still alive?) >> or ehcache, etc. >> >> Felix >> >> >Regards >> > >> >On Wednesday, August 3, 2016, Philippe Mouawad >> ><[email protected]> >> >wrote: >> > >> >> Hello, >> >> >> >> We have a PR-221 which relies on Guava. >> >> >> >> There is a discussion on github on the dependency on guava. >> >> >> >> What do team members think about this: >> >> - Should we introduce guava (which can be used for other things by >> >the way >> >> not only for this feature) >> >> - Should we use https://github.com/ben-manes/concurrentlinkedhashmap >> >> - Should we migrate to Java 8 and then use >> >https://github.com/ben-manes/ >> >> caffeine >> >> >> >> >> >> >> >> I vote for guava + PR integration as : >> >> >> >> - I see interesting things in Guava for JMeter even if we move to >> >Java >> >> 8: >> >> - cache >> >> - annotations >> >> - collections >> >> - ... >> >> >> >> Although it's a minor agurment, the PR is ready to go, it's not if we >> >> select CLHM or caffeine. >> >> >> >> >> >> >> >> >> >> -- >> >> Regards. >> >> Philippe >> >> >> >> > > > -- > Cordialement. > Philippe Mouawad.
