Skip to content

Conversation

@justinuang
Copy link
Contributor

…s running (googleapis#2529)

* Catch and log all exceptions in Watchdog runnable to ensure it never stops running

See https://stackoverflow.com/questions/6894595/scheduledexecutorservice-exception-handling for details

* runUnsafe
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 12, 2020
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

@igorbernstein2 igorbernstein2 merged commit 663bb8d into googleapis:bigtable-1.x May 12, 2020
@justinuang justinuang deleted the catch-watchdog-exceptions branch May 13, 2020 14:45
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.

3 participants