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

    https://github.com/apache/geode/pull/421#discussion_r105441883
  
    --- Diff: 
geode-lucene/src/main/java/org/apache/geode/cache/lucene/LuceneService.java ---
    @@ -37,15 +36,17 @@
      * </p>
      * <p>
      * Lucene indexes can be created using gfsh, xml, or the java API. Below 
is an example of creating a
    - * Lucene index with the java API. The Lucene index created on each member 
that will host data for
    - * the region.
    + * Lucene index with the java API. The Lucene index should be created on 
each member that has the
    + * region that is being indexed.
      * </p>
      * 
      * <pre>
      * {
      *   &#64;code
    - *   LuceneIndex index =
    - *       luceneService.createIndex(indexName, regionName, "field1", 
"field2", "field3");
    +*       luceneService.createCreateIndexFactory()
    --- End diff --
    
    I don't like the name "createCreateIndexFactory". 2 create?


---
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