Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release-23.2: streamingccl: replace replication lag metric with a replicated time metric #115234

Merged
merged 1 commit into from
Nov 29, 2023

Commits on Nov 27, 2023

  1. streamingccl: replace replication lag metric with a replicated time m…

    …etric
    
    This patch removes the replication lag metric and dbconsole graph as it's wrong
    if more than one replication stream is running, if the job coordinator moves,
    or if the job is paused.
    
    This patch adds the `physical_replication.replicated_time_seconds` metric which
    tracks the replicated time as seconds since the unix epoch. The metric uses
    seconds instead of nanos because the the `time()` prom function returns the
    seconds since the unix epoch, allowing users to easily alert if
    `time()-physical_replication.replicated_time_seconds` is too large.
    
    Lastly, this patch removes 2 c2c metrics that seemed useless.
    
    Informs #113951
    
    Release note (ui change): this patch remotes the
    `physical_replication.frontier_lag_nanos` metric and dbconsole graph is it's
    sometimes wrong. This patch adds `physical_replication.replicated_time_seconds`
    metric which users may use for alerting.
    msbutler committed Nov 27, 2023
    Configuration menu
    Copy the full SHA
    74a39ba View commit details
    Browse the repository at this point in the history