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