Hi Torben,

Makes sense. While it isn't a full-up integration test, I was able to build and run most of the GeoMesa unit tests with Guava 18. There was only one slightly issue; in one module, there was a method used which is not present in Guava 18.

I think if we removed that from GeoMesa, we'd be able to work with Guava 11-18. The GeoMesa datastore jars/zips/extensions targeting GeoServer 2.8.x do not include Guava since Guava 17 is part of the GeoServer distribution.

Anyhow, having classloaders per extension might be necessary at some point, but I'd like to try and help avoid it;).

Cheers,

Jim


On 06/28/2016 07:51 PM, Torben Barsballe wrote:
Hi Jim

One thing we were looking at was trying to make sure that GeoMesa (using Guava 17 by way of Accumulo) and ElasticGeo (using Guava 18 by way of Elasticsearch) wouldn't cause conflicts if both extensions were installed. One suggestion was to try and have each extension run in its own class loader. However, if we want to have a different version of Guava for these two dependencies, then we could not have Guava in core GeoServer (else it would conflict with one of the extensions).

Torben

On Tue, Jun 28, 2016 at 3:43 PM, Jim Hughes <[email protected] <mailto:[email protected]>> wrote:

    Hi Jody,

    Thanks for the clarification; I didn't get it that the line was a
    suggestion to remove Guava.

    From a cursory look, it appears that GeoTools has few dependencies
    on Guava.  For GeoServer, it seems that GWC and other pieces want
    to be able to have a Guava cache.  I don't think there's a Java 8
    analog for those caching options...

    For my money, I'd be happy if GeoServer picked a version of Guava
    (or a range of versions) to aim for.  Some of the Guava
    functionality has stayed the same between versions, and modules
    which can leverage that might work out better together.  (That is,
    there'd be some utility in using more common Guava functionality
    rather than a feature which is only in one version.)

    Do you have more details about the dependency issues with
    GeoMesa?  (I do see where the ElasticGeo instructions have you
    ripping out Guava, etc.)

    Joda doesn't use Guava.  Bringing up GeoServer and GeoMesa
    dependencies made me think of the Joda updates.

    Cheers,

    Jim


    On 06/28/2016 06:13 PM, Jody Garnett wrote:
    As per meeting notes I would like to know if removing Guava is
    something we can do over the course of 2.10 development.

    So does Joda time use Guava as well?
    --
    Jody Garnett

    On 28 June 2016 at 12:31, Jim Hughes <[email protected]
    <mailto:[email protected]>> wrote:

        Hi all,

        Looks like there is plenty going on, I just wanted to add
        some info re: GeoMesa and Guava.

        Since GeoMesa code ends up being deployed in GeoServer,
        Hadoop, and other environments, Guava is a provided
        dependency.  We try to use Guava features which have remained
        consistent between Guava versions (ranging from 11 to 17 or so).

        From talking to Tyler on GeoMesa's Gitter, we figured out
        that as GeoMesa moves to GT 15/GS 2.9, GeoMesa would need to
        update the Joda time dependency to 2.8 or 2.9.

        Let me know if there's a better practice or thing I can do to
        help out, etc.

        Cheers,

        Jim

        On 06/28/2016 12:34 PM, Jody Garnett wrote:

        Version hell:
        - Guava --> migrate to Java 8 (conflict with ElasticGeo and
        GeoMesa). Consder removing as.


        
------------------------------------------------------------------------------
        Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T
        Park in San
        Francisco, CA to explore cutting-edge tech and listen to tech
        luminaries
        present their vision of the future. This family event has
        something for
        everyone, including kids. Get more information and register
        today.
        http://sdm.link/attshape
        _______________________________________________
        Geoserver-devel mailing list
        [email protected]
        <mailto:[email protected]>
        https://lists.sourceforge.net/lists/listinfo/geoserver-devel




    
------------------------------------------------------------------------------
    Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T Park
    in San
    Francisco, CA to explore cutting-edge tech and listen to tech
    luminaries
    present their vision of the future. This family event has
    something for
    everyone, including kids. Get more information and register today.
    http://sdm.link/attshape
    _______________________________________________
    Geoserver-devel mailing list
    [email protected]
    <mailto:[email protected]>
    https://lists.sourceforge.net/lists/listinfo/geoserver-devel



------------------------------------------------------------------------------
Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T Park in San
Francisco, CA to explore cutting-edge tech and listen to tech luminaries
present their vision of the future. This family event has something for
everyone, including kids. Get more information and register today.
http://sdm.link/attshape
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to