You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by qiuchenjian <gi...@git.apache.org> on 2019/01/04 09:17:11 UTC

[GitHub] carbondata pull request #3050: ���CARBONDATA-3211��� Optimize the documentat...

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

    https://github.com/apache/carbondata/pull/3050#discussion_r245244853
  
    --- Diff: datamap/lucene/src/main/java/org/apache/carbondata/datamap/lucene/LuceneFineGrainDataMapFactory.java ---
    @@ -57,7 +57,7 @@ public LuceneFineGrainDataMapFactory(CarbonTable carbonTable, DataMapSchema data
               DataMapWriter.getDefaultDataMapPath(tableIdentifier.getTablePath(),
                   segment.getSegmentNo(), dataMapName), segment.getConfiguration()));
         } catch (MemoryException e) {
    -      LOGGER.error("failed to get lucene datamap , detail is {}" + e.getMessage());
    +      LOGGER.error("failed to get lucene datamap, detail is {}" + e.getMessage());
    --- End diff --
    
    ```suggestion
          LOGGER.error(String.format("failed to get lucene datamap, detail is %s", e.getMessage()));
    ```
    I think this log can't print the exception message


---