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

    https://github.com/apache/incubator-rya/pull/256#discussion_r159312925
  
    --- Diff: 
extras/indexing/src/main/java/org/apache/rya/indexing/statement/metadata/matching/RyaQueryEngineFactory.java
 ---
    @@ -64,12 +60,13 @@
                     throw new RuntimeException(e);
                 }
                 return (RyaQueryEngine<C>) new AccumuloRyaQueryEngine(conn, 
aConf);
    -        } else if(conf instanceof MongoDBRdfConfiguration && 
conf.getBoolean("sc.useMongo", false)) {
    -            MongoClient client = 
MongoConnectorFactory.getMongoClient(conf);
    -            return (RyaQueryEngine<C>) new 
MongoDBQueryEngine((MongoDBRdfConfiguration) conf, client);
    +        } else if(conf instanceof StatefulMongoDBRdfConfiguration && 
conf.getBoolean("sc.useMongo", false)) {
    --- End diff --
    
    There's a constant for "sc.useMongo" inside ConfigUtils.  Maybe use that 
here or move that constant to MongoDBRdfConfiguration and use it.


---

Reply via email to