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

    https://github.com/apache/incubator-rya/pull/72#discussion_r75383666
  
    --- Diff: 
extras/indexing/src/main/java/mvm/rya/indexing/FilterFunctionOptimizer.java ---
    @@ -101,23 +102,23 @@ public void setConf(final Configuration conf) {
             init = false;
             try {
                 init();
    -        } catch (final NumberFormatException | UnknownHostException e) {
    +        } catch (final NumberFormatException | IOException e) {
                 LOG.error("Unable to update to use new config, falling back to 
the old config.", e);
                 init = true;
             }
         }
     
    -    private synchronized void init() throws NumberFormatException, 
UnknownHostException {
    +    private synchronized void init() throws NumberFormatException, 
IOException {
             if (!init) {
                 if (ConfigUtils.getUseMongo(conf)) {
                     try {
                         final MongoClient mongoClient = 
MongoConnectorFactory.getMongoClient(conf);
    -                    geoIndexer = new MongoGeoIndexer(mongoClient);
    -                    geoIndexer.setConf(conf);
    -                    freeTextIndexer = new 
MongoFreeTextIndexer(mongoClient);
    -                    freeTextIndexer.setConf(conf);
    -                    temporalIndexer = new 
MongoTemporalIndexer(mongoClient);
    -                    temporalIndexer.setConf(conf);
    +                    geoIndexer = new MongoGeoIndexer();
    +                    ((AbstractMongoIndexer) geoIndexer).initIndexer(conf, 
mongoClient);
    --- End diff --
    
    that might be a problem in the other initIndexer()  The reason there is a 
seperate initIndexer() function is because init() does things I don't want to 
happen quite yet.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to