Github user manishnalla1994 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3022#discussion_r244118775
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/store/CarbonFactDataHandlerModel.java
---
@@ -356,9 +359,20 @@ public static CarbonFactDataHandlerModel
getCarbonFactDataHandlerModel(CarbonLoa
.getColumnSchemaList(carbonTable.getDimensionByTableName(tableName),
carbonTable.getMeasureByTableName(tableName));
carbonFactDataHandlerModel.setWrapperColumnSchema(wrapperColumnSchema);
- // get the cardinality for all all the columns including no dictionary
columns
- int[] formattedCardinality = CarbonUtil
-
.getFormattedCardinality(segmentProperties.getDimColumnsCardinality(),
wrapperColumnSchema);
+ // get the cardinality for all all the columns including no
+ // dictionary columns and complex columns
+ int[] dimAndComplexColumnCardinality =
+ new int[segmentProperties.getDimColumnsCardinality().length +
segmentProperties
+ .getComplexDimColumnCardinality().length];
+ for (int i = 0; i <
segmentProperties.getDimColumnsCardinality().length; i++) {
+ dimAndComplexColumnCardinality[i] =
segmentProperties.getDimColumnsCardinality()[i];
--- End diff --
The restructure case is not handled for complex types compaction. I have
raised the JIRA issue and I will handle it. Please find the JIRA link :
'https://issues.apache.org/jira/browse/CARBONDATA-3203' .
---
