From a88c64908891c4ba203aec7c8dc6f9444b5c7ac0 Mon Sep 17 00:00:00 2001 From: Ruihang Xia Date: Tue, 15 Nov 2022 11:17:42 +0800 Subject: [PATCH] fix: force set gRPC create request's table ID to None (#502) * fix: force set gRPC create request's table ID to None Signed-off-by: Ruihang Xia * fix: fix style Signed-off-by: Ruihang Xia Signed-off-by: Ruihang Xia --- src/frontend/src/instance.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/frontend/src/instance.rs b/src/frontend/src/instance.rs index 2f0aae5438..034b9ac38c 100644 --- a/src/frontend/src/instance.rs +++ b/src/frontend/src/instance.rs @@ -650,7 +650,12 @@ fn get_schema_name(expr: &AdminExpr) -> &str { #[async_trait] impl GrpcAdminHandler for Instance { - async fn exec_admin_request(&self, expr: AdminExpr) -> server_error::Result { + async fn exec_admin_request(&self, mut expr: AdminExpr) -> server_error::Result { + // Force the default to be `None` rather than `Some(0)` comes from gRPC decode. + // Related issue: #480 + if let Some(api::v1::admin_expr::Expr::Create(create)) = &mut expr.expr { + create.table_id = None; + } self.admin(get_schema_name(&expr)) .do_request(expr.clone()) .await