Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1626#discussion_r155263690
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/AggregationDataMapSchema.java
---
@@ -74,7 +86,49 @@ public ColumnSchema
getNonAggChildColBasedByParent(String columnName) {
Iterator<ColumnSchema> iterator = columnSchemas.iterator();
while (iterator.hasNext()) {
ColumnSchema next = iterator.next();
- if (null == next.getAggFunction() ||
next.getAggFunction().isEmpty()) {
+ if ((null == next.getAggFunction() ||
next.getAggFunction().isEmpty()) && null == next
--- End diff --
please rename method to `getNonAggNonTimeChildColBasedByParent`
---
