[GitHub] carbondata pull request #2019: [CARBONDATA-2172][Lucene] Add text_columns pr...

[jackylk]

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

    https://github.com/apache/carbondata/pull/2019#discussion_r171568299
  
    --- Diff: 
datamap/lucene/src/main/java/org/apache/carbondata/datamap/lucene/LuceneDataMapFactoryBase.java
 ---
    @@ -133,6 +120,44 @@ public void init(AbsoluteTableIdentifier identifier, 
DataMapSchema dataMapSchema
         analyzer = new StandardAnalyzer();
       }
     
    +  private List<String> validateAndGetIndexedColumns(DataMapSchema 
dataMapSchema,
    --- End diff --
    
    add comment, what is validated


---