Skip to content

Fix for Query Serving when Toposerver is Down#7484

Merged
harshit-gangal merged 7 commits intovitessio:masterfrom
planetscale:consul-qs-fix
Feb 17, 2021
Merged

Fix for Query Serving when Toposerver is Down#7484
harshit-gangal merged 7 commits intovitessio:masterfrom
planetscale:consul-qs-fix

Conversation

@harshit-gangal
Copy link
Member

@harshit-gangal harshit-gangal commented Feb 11, 2021

Description

Related Issue(s)

Checklist

  • Should this PR be backported?
  • 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

harshit-gangal and others added 3 commits February 11, 2021 18:13
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
@systay systay marked this pull request as ready for review February 15, 2021 07:39
@systay systay requested review from sougou and systay as code owners February 15, 2021 07:40
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Copy link
Collaborator

@deepthi deepthi left a comment

Choose a reason for hiding this comment

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

Mostly looks good. Just a couple of minor questions/comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vitess returns error when Consul topology service is unavailable

4 participants