Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1471#discussion_r151598459
--- Diff:
core/src/main/java/org/apache/carbondata/core/datamap/dev/AbstractDataMapWriter.java
---
@@ -0,0 +1,110 @@
+/*
--- End diff --
Can you explain , why change "DataMapWriter.java" to
"AbstractDataMapWriter.java", for easier supporting uses to customize other
type of datamapwriter?---
