Skip to content

Conversation

@elisheva-qlogic
Copy link
Contributor

Fixes #3815
Updated comments in:
BigtableInstanceAdminClient
BigtableTableAdminClient
BigtableDataClient

@elisheva-qlogic elisheva-qlogic requested a review from a team as a code owner October 18, 2018 19:23
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 18, 2018
Copy link

@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.

Thanks for cleaning this up. LGTM, but please remove the exists() code

}
}, MoreExecutors.directExecutor());
}

This comment was marked as spam.

@elisheva-qlogic elisheva-qlogic force-pushed the BigtableDataClient_comment_update branch from 9e04a1e to 2863e50 Compare October 22, 2018 14:49
@sduskis sduskis added the api: bigtable Issues related to the Bigtable API. label Oct 22, 2018
@igorbernstein2 igorbernstein2 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 23, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 23, 2018
@igorbernstein2 igorbernstein2 merged commit 64c0ee6 into googleapis:master Oct 23, 2018
@JesseLovelace JesseLovelace mentioned this pull request Oct 25, 2018
@elisheva-qlogic elisheva-qlogic deleted the BigtableDataClient_comment_update branch November 8, 2018 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants