CalvinKirs commented on code in PR #68176:
URL: https://github.com/apache/doris/pull/68176#discussion_r4045216488
##########
gensrc/thrift/FrontendService.thrift:
##########
@@ -906,6 +906,7 @@ enum TSchemaTableName {
SQL_BLOCK_RULE_STATUS = 12,
AUTHENTICATION_INTEGRATIONS = 13,
ROLE_MAPPINGS = 14,
+ EXTENSIONS = 18,
Review Comment:
Note on the thrift enum values: `EXTENSIONS = 18` and `SCH_EXTENSIONS = 73`
intentionally match master instead of taking the next free number on
branch-4.2. `TSchemaTableName` is sent from BE to FE, so using 15 here would
collide with master's `TABLE_STREAMS = 15` during a rolling upgrade. Gaps in a
thrift enum are fine.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
