-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat(bigtable): Add UpdateFamily to allow updating a family type #10759
Conversation
Please resolve vet failures |
Please add description to the PR |
ignoreWarnings bool | ||
} | ||
|
||
// GCPolicyOption is the interface to change GC policy settings | ||
type GCPolicyOption interface { |
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.
Ideally, we try to avoid modifying already exported and released types. This would break customer's code
https://pkg.go.dev/cloud.google.com/go/bigtable#GCPolicyOption
But I would like to know what any of the admins think
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.
I agree. I would suggest just updating the documentation.
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.
Done.
done |
Done |
It is currently only possible to update a column GC policy. We want to be able to also update the value_type of a column family.