Skip to content

Conversation

@rahulKQL
Copy link
Contributor

@rahulKQL rahulKQL commented Jan 7, 2019

What changes it contains:

Adding interface for admin method. Considered all operations present in grpc.BigtableTableAdminClient to update to equivalent v2.BigtableTableAdminClient.

Would consider two step migration to v2.BigtableTableAdminClient:

  1. Update all implementation to adapt to gax models.
  2. Update async methods to adapt to ApiFuture.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 7, 2019
Copy link
Collaborator

@igorbernstein2 igorbernstein2 left a comment

Choose a reason for hiding this comment

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

LGTM

@sduskis sduskis merged commit c00931d into googleapis:master Jan 8, 2019
@rahulKQL rahulKQL deleted the admin_method_changes branch January 9, 2019 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants