Github user kumarvishal09 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1804#discussion_r161509204
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
    @@ -2315,7 +2315,10 @@ public static boolean 
hasTimeSeriesDataMap(CarbonTable carbonTable) {
         List<DataMapSchema> dataMapSchemaList = 
carbonTable.getTableInfo().getDataMapSchemaList();
         for (DataMapSchema dataMapSchema : dataMapSchemaList) {
           if (dataMapSchema instanceof AggregationDataMapSchema) {
    -        return ((AggregationDataMapSchema) 
dataMapSchema).isTimeseriesDataMap();
    +        boolean istimeseries = ((AggregationDataMapSchema) 
dataMapSchema).isTimeseriesDataMap();
    --- End diff --
    
    Please do not store isTimeseriesDatamap value in any boolean add a if check 
and return false


---

Reply via email to