-
Notifications
You must be signed in to change notification settings - Fork 157
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
Cleanup of scyllaclient
package
#1309
Conversation
In favor of github.com/scylladb/scylladb-swagger-go-client
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, module the replace (the repo has already been transferred)
df633f6
to
6b6decc
Compare
scyllaclient package was cleaned from unused code that we inherited by copy pasting entire package from Scylla Manager. Summary of changes: * Autogenerated raw swagger client is now taken from github.com/scylladb/scylladb-swagger-go-client. * Client logging is now unified with our project logging library. * Datacenter cache was removed - we don't use Datacenter information anywhere. * Unused functions were removed. * Unused transport middlewares were removed. * Unused config fields were removed. * Added Config validation.
6b6decc
to
60256dc
Compare
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.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tnozicka, zimnx The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
scyllaclient package was cleaned from unused code that we inherited by
copy pasting entire package from Scylla Manager.
Summary of changes:
Prerequisites:
scylladb-swagger-go-client
repository transfer is finished