Skip to content

delete RPCs that were deprecated in 13.0#10099

Merged
deepthi merged 4 commits intovitessio:mainfrom
planetscale:ds-del-rpcs
Apr 20, 2022
Merged

delete RPCs that were deprecated in 13.0#10099
deepthi merged 4 commits intovitessio:mainfrom
planetscale:ds-del-rpcs

Conversation

@deepthi
Copy link
Copy Markdown
Collaborator

@deepthi deepthi commented Apr 15, 2022

Description

Delete all references to the RPCs listed in the linked issue

Related Issue(s)

Fixes #8839

Checklist

  • "Backport me!" label has been added if this change should be backported
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Signed-off-by: deepthi <deepthi@planetscale.com>
@mattlord
Copy link
Copy Markdown
Member

Reading through this again: https://github.com/vitessio/enhancements/blob/main/veps/vep-1.md#example-2-deprecating-and-removing-an-obsolete-utility

Did we return an error e.g. if someone hit this code via a cmd-line command or API call? Does that matter in this case?

@deepthi
Copy link
Copy Markdown
Collaborator Author

deepthi commented Apr 15, 2022

Reading through this again: https://github.com/vitessio/enhancements/blob/main/veps/vep-1.md#example-2-deprecating-and-removing-an-obsolete-utility

Did we return an error e.g. if someone hit this code via a cmd-line command or API call? Does that matter in this case?

The older versions of the RPCs are only called from the command-line of older version vtctlds. We have no way of going back and printing a warning from there.
If they upgrade their vtctld to a newer version (13+), it uses the new RPCs.
The 2-version process we follow for RPCs is meant to make this completely transparent to users, so we don't even try to print a warning. IOW, as long as you stay +/-1 version among the various vitess components, there will be no errors. If you have version drift beyond that, then there are no guarantees. You will in fact get an error.

I don't think we officially support direct RPC calls to vttablets from clients right now, so we don't worry about that.

Copy link
Copy Markdown
Member

@mattlord mattlord left a comment

Choose a reason for hiding this comment

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

I had a question about the vttablet fuzzer as there we seem to be removing rather than renaming/replacing, but otherwise this LGTM.

Signed-off-by: deepthi <deepthi@planetscale.com>
Copy link
Copy Markdown
Member

@mattlord mattlord left a comment

Choose a reason for hiding this comment

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

Thanks again!

deepthi added 2 commits April 18, 2022 16:50
Signed-off-by: deepthi <deepthi@planetscale.com>
Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi merged commit 4cc5000 into vitessio:main Apr 20, 2022
@deepthi deepthi deleted the ds-del-rpcs branch April 20, 2022 18:58
timvaillancourt added a commit to slackhq/vitess that referenced this pull request Mar 28, 2023
timvaillancourt added a commit to slackhq/vitess that referenced this pull request Mar 30, 2023
timvaillancourt added a commit to slackhq/vitess that referenced this pull request Mar 31, 2023
`slack-vitess-r14.0.5`: Revert "delete RPCs that were deprecated in 13.0 (vitessio#10099)"
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.

Inclusive Naming: delete deprecated RPCs

2 participants