[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [geomesa-users] GeoServer/GeoTools Version
|
Hi,
I was able to update the basic Geomesa to GeoServer 2.7.1.1 and GeoTools 13.1. However, I ran into issues with the scala versions being set. Our project has some dependencies for scala 2.11.2. So, I set about updating GeoMesa to scala 2.11.2.
This caused a required update to the scala logging com.typesafe.scala-logging:scala-logging-slf4j_2.11 version 2.1.2
This cascaded to a required update to all the import statements for Logging and the extends Logging etc. classes. I updated all of this to the new LazyLogging in the scala-logging package.
Then, I had to add the dependency for the org.scala-lang.modules:scala-parser-combinators_2.11 version 1.0.4
However, I am now running into an issue with the org/locationtech/geomesa/convert/SimpleFeatureConverterFactory.scala class throwing
SimpleFeatureConverterFactory.scala:119: error: can't expand macros compiled by previous versions of Scala
The previous error thrown like this was due to the logging. The new rendition of the error is
[ERROR] reuse(i) = requiredFields(i).eval(t)
[ERROR] ^
So far, I have not been able to pinpoint what the underlying macro is that is being called to update an references. Any help is most appreciated.
Chris Snider
Senior Software Engineer
Intelligent Software Solutions, Inc.
-----Original Message-----
From: geomesa-users-bounces@xxxxxxxxxxxxxxxx [mailto:geomesa-users-bounces@xxxxxxxxxxxxxxxx] On Behalf Of Chris Eichelberger
Sent: Friday, July 31, 2015 11:12 AM
To: Geomesa User discussions <geomesa-users@xxxxxxxxxxxxxxxx>
Subject: 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.g
> eotools.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
_______________________________________________
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
