carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ravipesala <...@git.apache.org>
Subject [GitHub] carbondata pull request #1425: [WIP]Thrift Changes to support Pre-aggregate
Date Mon, 23 Oct 2017 04:10:27 GMT
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


---

Mime
View raw message