You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2022/07/01 10:13:22 UTC

[GitHub] [doris] yiguolei commented on a diff in pull request #10136: [Feature] Lightweight schema change of add/drop column

yiguolei commented on code in PR #10136:
URL: https://github.com/apache/doris/pull/10136#discussion_r911827461


##########
gensrc/proto/olap_file.proto:
##########
@@ -93,6 +93,8 @@ message RowsetMetaPB {
     optional int64 num_segments = 22;
     // rowset id definition, it will replace required rowset id 
     optional string rowset_id_v2 = 23;
+    // tablet meta pb, for compaction
+    optional TabletSchemaPB tablet_schema = 24;

Review Comment:
   No need, there is already a tablet schema for every segment.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org