Skip to content

resurrect gateway interface and command line option#5992

Merged
sougou merged 2 commits intovitessio:masterfrom
planetscale:ds-gateway-interface
Mar 31, 2020
Merged

resurrect gateway interface and command line option#5992
sougou merged 2 commits intovitessio:masterfrom
planetscale:ds-gateway-interface

Conversation

@deepthi
Copy link
Copy Markdown
Collaborator

@deepthi deepthi commented Mar 31, 2020

This PR reverts a portion of #5978
It brings back the Gateway interface and the command line flag gateway_implementation.

There will shortly be two new Gateway implementations:

Some of the changes in #5978 are retained: deleting gateway package, and having discoverygateway tests use the concrete type instead of the interface.

Signed-off-by: deepthi deepthi@planetscale.com

Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi requested a review from sougou as a code owner March 31, 2020 01:56
Copy link
Copy Markdown
Contributor

@sougou sougou left a comment

Choose a reason for hiding this comment

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

One minor nit.

Signed-off-by: deepthi <deepthi@planetscale.com>
@sougou sougou merged commit 5072744 into vitessio:master Mar 31, 2020
@deepthi deepthi deleted the ds-gateway-interface branch May 14, 2020 16:47
@frouioui frouioui mentioned this pull request Mar 9, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants