-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
server: Add table range api to http status server (#20456) #20620
server: Add table range api to http status server (#20456) #20620
Conversation
Signed-off-by: Xiaoguang Sun <sunxiaoguang@zhihu.com>
@lonng Could you please check this cherry pick so it can be included in the next 4.0 release |
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
@sunxiaoguang Sorry for the late review. |
@Yisaer PTAL |
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 |
/run-all-tests |
Cherry pick pr #20456
Signed-off-by: Xiaoguang Sun sunxiaoguang@zhihu.com
What problem does this PR solve?
Add table range api to http status server so users can easily query misc ranges information of a particular table.
What is changed and how it works?
Add /tables/{db}/{table}/ranges to http status server and return ranges information row and all indices of the table.
What's Changed:
Add /tables/{db}/{table}/ranges to http status server.
How it Works:
Compute row ranges based on table's id, indices ranges from table's metadata.
Related changes
Add routing information to server/http_status.go
Add handler to server/http_handler.go
Tests
server/http_handler_test.go
TestRangesAPI
TestListTableRanges
Side effects
None
Release note