-
Notifications
You must be signed in to change notification settings - Fork 144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add join proto #171
add join proto #171
Conversation
Signed-off-by: Fei Han <hanfei19910905@gmail.com> fix adjust join proto refine proto refine proto refine proto refine proto refine proto refine proto refine proto refine proto refine proto refine proto add key range for table scan change go code
update protobuf-build to 0.11 and prost to 0.6 update ci image Signed-off-by: Jay Lee <BusyJayLee@gmail.com>
Signed-off-by: fredchenbj <cfworking@163.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Feng Liyuan <darktemplar.f@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
proto/executor.proto
Outdated
} | ||
|
||
message TableScan { | ||
optional int64 table_id = 1 [(gogoproto.nullable) = false]; | ||
repeated ColumnInfo columns = 2; | ||
optional bool desc = 3 [(gogoproto.nullable) = false]; | ||
repeated int64 primary_column_ids = 4; | ||
optional EngineType next_read_engine = 5 [(gogoproto.nullable) = false]; // which engine we should in next step, only used by tiflash | ||
repeated KeyRange ranges = 6 [(gogoproto.nullable) = false]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please document this field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/merge |
Signed-off-by: Fei Han hanfei19910905@gmail.com