QiangCai commented on a change in pull request #3661: [CARBONDATA-3704] Support 
create materialized view on all type table, and make mv support mutil-tenant.
URL: https://github.com/apache/carbondata/pull/3661#discussion_r401314484
 
 

 ##########
 File path: 
core/src/main/java/org/apache/carbondata/core/datamap/DataMapProvider.java
 ##########
 @@ -48,20 +48,20 @@
 import org.apache.log4j.Logger;
 
 /**
- * DataMap is a accelerator for certain type of query. Developer can add new 
DataMap
+ * Index is a accelerator for certain type of query. Developer can add new 
Index
  * implementation to improve query performance.
  *
- * Currently two types of DataMap are supported
+ * Currently two types of Index are supported
  * <ol>
- *   <li> MVDataMap: materialized view type of DataMap to accelerate olap 
style query,
+ *   <li> MVDataMap: materialized view type of Index to accelerate olap style 
query,
  * like SPJG query (select, predicate, join, groupby) </li>
- *   <li> DataMap: index type of DataMap to accelerate filter query </li>
+ *   <li> Index: index type of Index to accelerate filter query </li>
  * </ol>
  *
  * <p>
  * In following command <br>
  * {@code CREATE DATAMAP dm ON TABLE main USING 'provider'}, <br>
 
 Review comment:
   do we still have this SQL syntax?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to