-
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
http: add http api for get db and table info that is related to the tableID #8256
Conversation
@zanmato1984 PTAL |
LGTM. I think you might need another approve from some TiDB team member? |
I think we could get the table info by Line 52 in f497444
So could we append some DB info to the old API? btw: when we get the table info, I think we could get the DB info in the other way. I'm not sure why we need to append the DB info in the same API. |
f527d35
to
ceb102c
Compare
I think to add a new api is better, if you change the return format, will cause compatibility issues. |
14706fa
to
d30d7d9
Compare
d30d7d9
to
22103f6
Compare
@winkyao 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
@winkyao 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
…ableID (pingcap#8256) * http: add http api for get db and table info that is related to the tableID
What problem does this PR solve?
add http api to get db and table info that is related to the tableID.
This is for the flash use.
This will return below now:
What is changed and how it works?
Check List
Tests
Code changes
Side effects
Related changes
This change is