[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [geomesa-users] GeoServer/GeoTools Version
|
Chris,
Upgrading GeoServer and GeoTools dependencies is on our list, but we
haven't gotten there yet. (Part of our intellectual property review
being conducted by LocationTech will necessitate some of these
upgrades).
If I follow your proposal correctly, then you are talking about removing
these lines (that you quote):
https://github.com/locationtech/geomesa/blob/master/geomesa-filter/src/main/scala/org/locationtech/geomesa/filter/expression/FastPropertyName.scala#L45-L50
Flipping through the rest of the GeoMesa source, it looks as if --
independent of the fact that GeoTools has deprecated those methods -- we
may be using them elsewhere.
Feel free to give this fix a try, but I suspect you would run into other
build errors from some of the other GeoMesa modules. Let us know what
you find!
Sincerely,
-- Chris
On Fri, 2015-07-31 at 16:54 +0000, Chris Snider wrote:
> Hi,
>
>
>
> I grabbed the Geomesa repo and checked out the master branch. I
> updated the master pom to user geoserver.version 2.7.1.1 which
> requires the change to geotools.version 13.1.
>
>
>
> I tried to compile, but get errors in
> org/locationtech/geomesa/filter/expression/FastPropertyName.scala due
> to the updates in GeoTools for
> theorg.geotools.filter.expression.FilterVisitorExpressionWrapper,org.geotools.filter.Expression andorg.geotools.filter.FilterVisitor
>
>
>
> The FilterVistorExpressionWrapper was removed in GeoTools 12.x with no
> replacement that I have identified so far.
>
>
>
> org.geotools.filter.Expression appears to be replaced
> withorg.opengis.filter.expression.Expression
>
>
>
> org.geotools.filter.FilterVisitor appears to be replaced
> withorg.opengis.filter.FilterVisitor
>
>
>
> It looks like I should be able to remove the import org.geotools.filter.expression.FilterVisitorExpressionWrapper statement then remove the block at the bottom of the class represented by
>
> // geotools filter methods - deprecated but still sometimes used
> override defgetType: Short = ExpressionType.ATTRIBUTE
> override defgetValue(feature: SimpleFeature): AnyRef =
> evaluate(feature.asInstanceOf[AnyRef])
> override defevaluate(feature: SimpleFeature): AnyRef =
> evaluate(feature.asInstanceOf[AnyRef])
> override defaccept(visitor: FilterVisitor): Unit =
> accept(newFilterVisitorExpressionWrapper(visitor),null)
>
>
>
> Does this look accurate?
>
>
>
> Chris Snider
>
> Senior Software Engineer
>
> Intelligent Software Solutions, Inc.
>
> Description: Description: Description:
> cid:image001.png@01CA1F1F.CBC93990
>
>
>
>
> From: geomesa-users-bounces@xxxxxxxxxxxxxxxx
> [mailto:geomesa-users-bounces@xxxxxxxxxxxxxxxx] On Behalf Of Jim
> Hughes
> Sent: Friday, July 17, 2015 11:29 AM
> To: geomesa-users@xxxxxxxxxxxxxxxx
> Subject: Re: [geomesa-users] GeoServer/GeoTools Version
>
>
>
>
> Chris,
>
> I'd definitely suggest building from master. We've made some
> performance enhancements in the 1.1.0-rc.x series.
>
> Cheers,
>
> Jim
>
> On 07/17/2015 01:21 PM, Chris Snider wrote:
>
>
> Jim,
>
>
>
> Should I build Geomesa from the Accumulo1.5/1.x branch, or
> should I build it from Master?
>
>
>
> Thanks,
>
>
>
> Chris Snider
>
> Senior Software Engineer
>
> Intelligent Software Solutions, Inc.
>
> Description: Description: Description:
> cid:image001.png@01CA1F1F.CBC93990
>
>
>
>
> From:geomesa-users-bounces@xxxxxxxxxxxxxxxx
> [mailto:geomesa-users-bounces@xxxxxxxxxxxxxxxx] On Behalf Of
> Jim Hughes
> Sent: Friday, July 17, 2015 11:13 AM
> To: geomesa-users@xxxxxxxxxxxxxxxx
> Subject: Re: [geomesa-users] GeoServer/GeoTools Version
>
>
>
>
> Hi Chris,
>
> Yes, it should. You should just have to update the gt.version
> tag in the root pom.xml to 13.1.
>
> I'd suggest running the unit tests; if you hit any errors,
> feel free to send them along and I'll help out.
>
> Cheers,
>
> Jim
>
> On 07/17/2015 12:50 PM, Chris Snider wrote:
>
>
> Hi,
>
>
>
> Will Geomesa compile and/or run against an
> installation of GeoServer 2.7.1.1 or 2.6.4 ?
>
>
>
> Thanks,
>
>
>
> Chris Snider
>
> Senior Software Engineer
>
> Intelligent Software Solutions, Inc.
>
> Description: Description: Description:
> cid:image001.png@01CA1F1F.CBC93990
>
>
>
>
>
>
>
> _______________________________________________
> geomesa-users mailing list
> geomesa-users@xxxxxxxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> http://www.locationtech.org/mailman/listinfo/geomesa-users
>
>
>
>
>
>
> _______________________________________________
> geomesa-users mailing list
> geomesa-users@xxxxxxxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> http://www.locationtech.org/mailman/listinfo/geomesa-users
>
>
>
>
> _______________________________________________
> geomesa-users mailing list
> geomesa-users@xxxxxxxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> http://www.locationtech.org/mailman/listinfo/geomesa-users
