Skip to content

Commit

Permalink
fix!: change request id from int to string (#127)
Browse files Browse the repository at this point in the history
  • Loading branch information
jiacai2050 authored Jan 10, 2024
1 parent 4a6f323 commit 19ece8f
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions protos/engine/remote_engine.proto
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ enum ReadOrder {
}

message TableReadRequest {
uint64 request_id = 1;
string request_id = 1;
ReadOptions opts = 2;
schema.ProjectedSchema projected_schema = 3;
Predicate predicate = 4;
Expand Down Expand Up @@ -167,16 +167,14 @@ enum QueryPriority {
enum Explain { Analyze = 0; }

message ExecContext {
// This field is deprecated, use request_id_str instead.
uint64 request_id = 1;
string request_id = 1;
string default_catalog = 2;
string default_schema = 3;
// -1 means no timeout
int64 timeout_ms = 4;
QueryPriority priority = 5;
string displayable_query = 6;
string request_id_str = 7;
optional Explain explain = 8;
optional Explain explain = 7;
}

message ExecutePlanRequest {
Expand Down

0 comments on commit 19ece8f

Please sign in to comment.