Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bigtable: Admin should have proto utility methods #3141

Closed
sduskis opened this issue Apr 10, 2018 · 3 comments
Closed

Bigtable: Admin should have proto utility methods #3141

sduskis opened this issue Apr 10, 2018 · 3 comments
Assignees
@igorbernstein2
Labels
api: bigtable Issues related to the Bigtable API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@sduskis
Copy link
Contributor

sduskis commented Apr 10, 2018

There are some complex protobuf objects. Here are some possibilities where utilities might come in handy:

  • any message with "oneof" gets confusjng, since users set multiple values (AppProfile, DropRowRangeRequest, ModifyColumnFamiliesRequest)
  • Instance messages are complex
  • PartialUpdateInstanceRequest use google.protobuf.FieldMask which is quite confusing
  • CheckConsistency is used for for polling, so it's ideal having a utility for this kind of functionality
  • "TableExists" type functionality should use ListTables instead of GetTable, since it's less expensive. Users should have a clear API that produces the most efficient calls.
@pongad pongad added the api: bigtable Issues related to the Bigtable API. label Apr 10, 2018
@JustinBeckwith JustinBeckwith added triage me I really want to be triaged. 🚨 This issue needs some love. labels Jun 8, 2018
@yihanzhen yihanzhen added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. priority: p2 Moderately-important priority. Fix may not be included in next release. and removed 🚨 This issue needs some love. triage me I really want to be triaged. labels Jun 20, 2018
@yihanzhen
Copy link
Contributor

I'm not sure about priority. Feel free to bump if necessary.

@igorbernstein2
Copy link

This is currently being worked on:
#3395

@igorbernstein2
Copy link

I'm still in the process of cleaning things up, but the wrappers have landed.

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

@igorbernstein2 igorbernstein2

Labels
api: bigtable Issues related to the Bigtable API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@sduskis @JustinBeckwith @pongad @igorbernstein2 @yihanzhen