Github user sujith71955 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/200#discussion_r85957010
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/graphgenerator/GraphGenerator.java
---
@@ -998,4 +1000,24 @@ private void
prepareIsUseInvertedIndex(List<CarbonDimension> dims,
graphConfig.setIsUseInvertedIndex(
isUseInvertedIndexList.toArray(new
Boolean[isUseInvertedIndexList.size()]));
}
+
+ /**
+ * Preparing the boolean [] to map whether the dimension use trim or not.
+ *
+ * @param dims
+ * @param graphConfig
+ */
+ private void prepareIsUseTrim(List<CarbonDimension> dims,
+ GraphConfigurationInfo graphConfig) {
+ List<Boolean> isUseTrimList = new ArrayList<Boolean>();
+ for (CarbonDimension dimension : dims) {
+ if (dimension.isUseTrim()) {
--- End diff --
Can we add this trim option as property of the column, i mean inside a
column property rather than setting directly as CarbonDimension property,
Already in CarbonColumn there is a property map which holds column related
properties, i think we can use that, please check the feasibility.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
