Skip to content

Bigtable: fix instance leaks from snippets#7800

Merged
tseaver merged 3 commits intogoogleapis:masterfrom
sangramql:bigtable_snippets_instance_leak
Apr 30, 2019
Merged

Bigtable: fix instance leaks from snippets#7800
tseaver merged 3 commits intogoogleapis:masterfrom
sangramql:bigtable_snippets_instance_leak

Conversation

@sangramql
Copy link
Copy Markdown
Contributor

Fix issue
#7003

@sangramql sangramql requested a review from tseaver as a code owner April 24, 2019 21:40
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 24, 2019
@sangramql
Copy link
Copy Markdown
Contributor Author

@crwilcox @AVaksman @sduskis
Please review the code.

@tseaver tseaver added api: bigtable Issues related to the Bigtable API. testing kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Apr 29, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@tseaver tseaver changed the title Fix instances leak from snippets Bigtable: fix instance leaks from snippets Apr 29, 2019
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@tseaver tseaver merged commit 86f81d1 into googleapis:master Apr 30, 2019
parthea pushed a commit that referenced this pull request Nov 22, 2025
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. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants