From 5e42eb5ec63caf9f0b07e5ba42d7f2fddd20199f Mon Sep 17 00:00:00 2001 From: fys <40801205+Fengys123@users.noreply.github.com> Date: Wed, 16 Nov 2022 17:41:34 +0800 Subject: [PATCH] fix: field number of proto (#541) --- src/api/greptime/v1/database.proto | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/api/greptime/v1/database.proto b/src/api/greptime/v1/database.proto index cbefdf19da..e4b651f322 100644 --- a/src/api/greptime/v1/database.proto +++ b/src/api/greptime/v1/database.proto @@ -42,11 +42,11 @@ message InsertExpr { string table_name = 2; message Values { - repeated bytes values = 3; + repeated bytes values = 1; } oneof expr { - Values values = 4; + Values values = 3; // TODO(LFC): Remove field "sql" in InsertExpr. // When Frontend instance received an insertion SQL (`insert into ...`), it's anticipated to parse the SQL and @@ -55,12 +55,12 @@ message InsertExpr { // Then why the "sql" field exists here? It's because the Frontend needs table schema to create the values to insert, // which is currently not able to find anywhere. (Maybe the table schema is suppose to be fetched from Meta?) // The "sql" field is meant to be removed in the future. - string sql = 5; + string sql = 4; } /// The region number of current insert request. - uint32 region_number = 6; - map options = 7; + uint32 region_number = 5; + map options = 6; } // TODO(jiachun)