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

    https://github.com/apache/carbondata/pull/2113#discussion_r181662181
  
    --- Diff: 
datamap/lucene/src/main/java/org/apache/carbondata/datamap/lucene/LuceneFineGrainDataMapFactory.java
 ---
    @@ -62,7 +62,17 @@
       public List<FineGrainDataMap> getDataMaps(DataMapDistributable 
distributable,
           ReadCommittedScope readCommittedScope)
           throws IOException {
    -    return getDataMaps(distributable.getSegment(), readCommittedScope);
    +    List<FineGrainDataMap> lstDataMap = new ArrayList<>();
    +    FineGrainDataMap dataMap = new LuceneFineGrainDataMap(analyzer);
    +    String indexPath = ((LuceneDataMapDistributable) 
distributable).getIndexPath();
    +    try {
    +      dataMap.init(new DataMapModel(indexPath));
    +    } catch (MemoryException e) {
    +      LOGGER.error("failed to get lucene datamap , detail is {}" + 
e.getMessage());
    --- End diff --
    
    you can use String.format() to format the log message


---

Reply via email to