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