Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1626#discussion_r155267822
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/AggregationDataMapSchema.java
---
@@ -210,4 +291,45 @@ private void fillAggFunctionColumns(List<ColumnSchema>
listOfColumns) {
}
}
+ public boolean isTimeseriesDataMap() {
+ return isTimeseriesDataMap;
+ }
+
+ /**
+ * Below method is to support rollup during loading the data in pre
aggregate table
+ * In case of timeseries year level table data loading can be done using
month level table or any
+ * time series level below year level for example day,hour minute,
second.
+ * @TODO need to handle for pre aggregate table without timeseries
+ *
+ * @param aggregationDataMapSchema
+ * @return whether aggregation data map can be selected or not
+ */
+ public boolean canSelectForRollup(AggregationDataMapSchema
aggregationDataMapSchema) {
+ List<ColumnSchema> listOfColumns = childSchema.getListOfColumns();
--- End diff --
please handle `dummy-measure` scenario
---
