Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [geomesa-users] GeoServer/GeoTools Version

Hi Chris,

Good/bad news.  Updating Scala to 2.11.7 is also on the short list...

As a quick check, have you trying blow away the org/locationtech folders in your local ~/.m2/repository? I'm wondering if Maven is dragging in an old version of the geomesa-convert libraries (which were compiled with Scala 2.10).

If you have a branch you are willing to push to GitHub, I'm happy to pitch in and see if I notice anything obvious.

Thanks,

Jim

On 08/03/2015 05:24 PM, Chris Snider wrote:
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
_______________________________________________
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



Back to the top