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

    https://github.com/apache/geode/pull/479#discussion_r113630711
  
    --- Diff: 
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneServiceImpl.java
 ---
    @@ -166,28 +166,28 @@ public void createIndex(final String indexName, 
String regionPath, final Analyze
        * 
        * Public because this is called by the Xml parsing code
        */
    -  public void afterDataRegionCreated(final String indexName, final 
Analyzer analyzer,
    -      final String dataRegionPath, final Map<String, Analyzer> 
fieldAnalyzers,
    -      final String... fields) {
    -    LuceneIndexImpl index = createIndexRegions(indexName, dataRegionPath);
    -    index.setSearchableFields(fields);
    -    index.setAnalyzer(analyzer);
    -    index.setFieldAnalyzers(fieldAnalyzers);
    +  public void afterDataRegionCreated(LuceneIndexImpl index) {
         index.initialize();
         registerIndex(index);
         if (this.managementListener != null) {
           this.managementListener.afterIndexCreated(index);
         }
    +
    +  }
    +
    +  public LuceneIndexImpl beforeDataRegionCreated(final String indexName, 
final String regionPath,
    +      RegionAttributes attributes, final Analyzer analyzer,
    +      final Map<String, Analyzer> fieldAnalyzers, String aeqId, final 
String... fields) {
    +    LuceneIndexImpl index = createIndexRegions(indexName, regionPath);
    --- End diff --
    
    Changed it to createIndexObject - any better suggestion is welcomed 


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