This is an automated email from the ASF dual-hosted git repository.
starocean999 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 170084d61e1 [fix](thrift) Reserve frontend service field ids (#63239)
170084d61e1 is described below
commit 170084d61e102ee36671d3e75d9bd35988c0e24c
Author: seawinde <[email protected]>
AuthorDate: Mon May 18 10:35:45 2026 +0800
[fix](thrift) Reserve frontend service field ids (#63239)
Reserve several unused `FrontendService.thrift` field ids for downstream
compatibility.
These ids are intentionally kept unused in Apache Doris so they are not
accidentally reused with different wire semantics in maintained branches
or downstream distributions.
No runtime behavior is changed. All added fields are optional reserved
fields and are not read or written by current code.
---
gensrc/thrift/FrontendService.thrift | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/gensrc/thrift/FrontendService.thrift
b/gensrc/thrift/FrontendService.thrift
index 07eb9333637..c47749de06b 100644
--- a/gensrc/thrift/FrontendService.thrift
+++ b/gensrc/thrift/FrontendService.thrift
@@ -65,6 +65,9 @@ struct TDescribeTablesParams {
5: optional Types.TUserIdentity current_user_ident // to replace the user
and user ip
6: optional bool show_hidden_columns = false
7: optional string catalog
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 8: optional set<string> reserved_field_8
}
// Results of a call to describeTable()
@@ -95,6 +98,9 @@ struct TGetDbsParams {
4: optional Types.TUserIdentity current_user_ident // to replace the user
and user ip
5: optional string catalog
6: optional bool get_null_catalog //if catalog is empty , get dbName
="NULL" and dbId = -1.
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 7: optional set<string> reserved_field_7
}
// getDbNames returns a list of database names , database ids and catalog
names ,catalog ids
@@ -119,6 +125,9 @@ struct TGetTablesParams {
6: optional string type
7: optional string catalog
8: optional string table
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 9: optional set<string> reserved_field_9
}
struct TTableStatus {
@@ -400,6 +409,11 @@ struct TMasterOpRequest {
32: optional bool moreResultExists // Server has more result to send
33: optional map<string, string> connect_attributes
+ // Reserved for downstream fields `current_roles` and `is_su_user` to keep
+ // thrift field ids wire-compatible across maintained branches. Do not
reuse these ids.
+ 34: optional set<string> reserved_field_34
+ 35: optional bool reserved_field_35
+
// selectdb cloud
1000: optional string cloud_cluster
1001: optional bool noAuth;
@@ -909,6 +923,9 @@ struct TMetadataTableRequestParams {
12: optional PlanNodes.TMetaCacheStatsParams meta_cache_stats_params
13: optional PlanNodes.TPartitionValuesMetadataParams
partition_values_metadata_params
14: optional PlanNodes.THudiMetadataParams hudi_metadata_params
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 15: optional set<string> reserved_field_15
}
struct TSchemaTableRequestParams {
@@ -920,6 +937,9 @@ struct TSchemaTableRequestParams {
6: optional string time_zone // used for DATETIME field
7: optional string frontend_conjuncts
8: optional i64 thread_id // mysql connection id for fetching
ConnectContext if needed
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 9: optional set<string> reserved_field_9
}
struct TFetchSchemaTableDataRequest {
@@ -1550,6 +1570,9 @@ struct TShowProcessListRequest {
1: optional bool show_full_sql
2: optional Types.TUserIdentity current_user_ident
3: optional string time_zone
+ // Reserved for downstream field `current_roles` to keep thrift field ids
+ // wire-compatible across maintained branches. Do not reuse this id.
+ 4: optional set<string> reserved_field_4
}
struct TShowProcessListResult {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
