Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1425#discussion_r146163549
--- Diff: format/src/main/thrift/schema.thrift ---
@@ -165,9 +165,46 @@ struct TableSchema{
4: optional map<string,string> tableProperties; // Table properties
configured by the user
5: optional BucketingInfo bucketingInfo; // Bucketing information
6: optional PartitionInfo partitionInfo; // Partition information
+ 7: optional list<ColumnRelation> columnRelationList;
}
+struct RelationIdentifier {
+ 1: optional string databaseName;
+ 2: required string tableName;
+ 3: required string tableId;
+}
+
+struct ColumnRelation{
+ // child column id
+ 1: required string columnId;
+ 2: optional list<ParentColumnTableRelation> columnTableRelation;
+}
+
+struct ParentColumnTableRelation {
+ 1: required RelationIdentifier relationInfo;
+ 2: required string columnId;
+ 3: required string columnName
+}
+
+enum RelationType {
+ AGGREGATION = 0;
+}
+struct RelationProperties {
+ 1: optional RelationType relationTypes;
--- End diff --
it is not required , but add the optional class name
---
