Skip to content

[vtadmin] Add FindSchema route#7610

Merged
rohit-nayak-ps merged 3 commits intovitessio:masterfrom
tinyspeck:am_vtadmin_find_schema
Mar 6, 2021
Merged

[vtadmin] Add FindSchema route#7610
rohit-nayak-ps merged 3 commits intovitessio:masterfrom
tinyspeck:am_vtadmin_find_schema

Conversation

@ajm188
Copy link
Contributor

@ajm188 ajm188 commented Mar 4, 2021

Description

This PR adds an endpoint to find a globally (cross-cluster) unique table schema in vtadmin.

Related Issue(s)

Checklist

  • Should this PR be backported? No
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

Andrew Mason added 3 commits March 4, 2021 07:59
Signed-off-by: Andrew Mason <amason@slack-corp.com>
Signed-off-by: Andrew Mason <amason@slack-corp.com>
Signed-off-by: Andrew Mason <amason@slack-corp.com>
@ajm188 ajm188 requested review from doeg and rohit-nayak-ps March 4, 2021 21:05
Copy link
Contributor

@doeg doeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. One non-blocking question.

@rohit-nayak-ps rohit-nayak-ps merged commit 2b18baa into vitessio:master Mar 6, 2021
@doeg doeg added the Component: VTAdmin VTadmin interface label Mar 16, 2021
@ajm188 ajm188 deleted the am_vtadmin_find_schema branch May 29, 2021 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: VTAdmin VTadmin interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add FindSchema route to vtadmin

3 participants