Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

[tablet] Split health_check_interval into two flags #248

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brirams
Copy link

@brirams brirams commented Jun 16, 2022

Description

The health_check_interval flag controls:

  • how often we send healthchecks to vtgates via the healthcheck stream
  • how often the replication manager checks the health of its replication stream

we'd like to change the first without affecting the other so this PR:

  • introduces the replication_health_check_interval for the latter
  • defaults the resolved config value to the same value as the passed in health_check_interval if the new flag is not provided
  • passed the new flag value into the TabletManager for use in constructing its replManager

Checklist

  • Tests were added or are not required
  • Documentation was added or is not required

Signed-off-by: Brian Ramos <brirams@users.noreply.github.com>
…_check_interval is being passed in

Signed-off-by: Brian Ramos <brirams@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant