[ 
https://issues.apache.org/jira/browse/RYA-324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16109847#comment-16109847
 ] 

ASF GitHub Bot commented on RYA-324:
------------------------------------

Github user isper3at commented on a diff in the pull request:

    https://github.com/apache/incubator-rya/pull/182#discussion_r130736014
  
    --- Diff: 
extras/rya.geoindexing/geo.common/src/main/java/org/apache/rya/indexing/accumulo/geo/OptionalConfigUtils.java
 ---
    @@ -104,26 +116,26 @@ public static void setIndexers(final 
RdfCloudTripleStoreConfiguration conf) {
     
             if (ConfigUtils.getUseMongo(conf)) {
                 if (getUseGeo(conf)) {
    -                if (geoIndexerType == null) {
    +                if (geoIndexerType == GeoIndexerType.UNSPECIFIED) {
                         // Default to MongoGeoIndexer if not specified
    -                    indexList.add(MongoGeoIndexer.class.getName());
    +                    
indexList.add(GeoIndexerType.MONGO_DB.getGeoIndexerClassString()); 
//MongoGeoIndexer.class.getName());
    --- End diff --
    
    remove commented out code


> geoWave and geoMesa depend on different versions of the GeoTools
> ----------------------------------------------------------------
>
>                 Key: RYA-324
>                 URL: https://issues.apache.org/jira/browse/RYA-324
>             Project: Rya
>          Issue Type: Bug
>          Components: sail
>    Affects Versions: 3.2.10
>            Reporter: David W. Lotts
>            Assignee: David W. Lotts
>              Labels: dependencies, geo, refactor
>             Fix For: 3.2.10
>
>
> Refactored geo libraries into separate projects/modules/jars.
> Geo libraries are optional and turned off by default because the GeoTools is 
> has an incompatible license.
> Further, geoWave and geoMesa depend on different versions of the GeoTools, so 
> they must be in separate projects.
> The new modules (projects) are
> geo.common
> geo.mongo
> geo.geomesa
> geo.geowave
> These are modules of the project rya.geoindexing



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to