Skip to content

changefeedccl: remove deprecated changefeed.min_high_water metric#36915

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
danhhz:cdc_metric
Apr 17, 2019
Merged

changefeedccl: remove deprecated changefeed.min_high_water metric#36915
craig[bot] merged 1 commit intocockroachdb:masterfrom
danhhz:cdc_metric

Conversation

@danhhz
Copy link
Copy Markdown
Contributor

@danhhz danhhz commented Apr 17, 2019

It was replaced by max_behind_nanos in 19.1.

Release note (backwards-incompatible change): Removed the deprecated
changefeed.min_high_water metric; use changefeed.max_behind_nanos
instead.

It was replaced by max_behind_nanos in 19.1.

Release note (backwards-incompatible change): Removed the deprecated
`changefeed.min_high_water` metric; use `changefeed.max_behind_nanos`
instead.
@danhhz danhhz requested review from a team, nvb and tbg April 17, 2019 17:45
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@danhhz
Copy link
Copy Markdown
Contributor Author

danhhz commented Apr 17, 2019

TFTR!

bors r=nvanbenschoten

craig bot pushed a commit that referenced this pull request Apr 17, 2019
36867: storage: take an engine checkpoint during failing consistency checks r=andreimatei a=tbg

This takes a checkpoint on the nodes with replicas of a failing range,
before the failure leads to nodes shutting down. The checkpoint will, for
the replicas of the affected range, be taken at the same Raft log position.

Release note: None

36915: changefeedccl: remove deprecated changefeed.min_high_water metric r=nvanbenschoten a=danhhz

It was replaced by max_behind_nanos in 19.1.

Release note (backwards-incompatible change): Removed the deprecated
`changefeed.min_high_water` metric; use `changefeed.max_behind_nanos`
instead.

Co-authored-by: Tobias Schottdorf <tobias.schottdorf@gmail.com>
Co-authored-by: Daniel Harrison <daniel.harrison@gmail.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 17, 2019

Build succeeded

@craig craig bot merged commit 019641d into cockroachdb:master Apr 17, 2019
@danhhz danhhz deleted the cdc_metric branch April 17, 2019 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants