-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Multiraft: snapshot/compaction integration #228
Milestone
Comments
@bdarnell is this still open? |
This is done. |
rohany
pushed a commit
to rohany/cockroach
that referenced
this issue
Sep 18, 2019
This PR adds telemetry for some of the new features added to cockroach demo (nodes, with-load, localities, geo-partitioned-replicas). It isn't straight forward how to test this, as there seems to be a race between the background thread that flushes telemetry and actually querying for the counter's values on startup. Work towards cockroachdb#228. Release justification: Low risk monitoring improvement. Release note: None
rohany
pushed a commit
to rohany/cockroach
that referenced
this issue
Sep 18, 2019
This PR adds telemetry for some of the new features added to cockroach demo (nodes, with-load, localities, geo-partitioned-replicas). It isn't straight forward how to test this, as there seems to be a race between the background thread that flushes telemetry and actually querying for the counter's values on startup. Work towards cockroachdb#228. Release justification: Low risk monitoring improvement. Release note: None
craig bot
pushed a commit
that referenced
this issue
Sep 18, 2019
40883: demo: Add telemetry about usage of new demo features. r=jordanlewis a=rohany This PR adds telemetry for some of the new features added to cockroach demo (nodes, with-load, localities, geo-partitioned-replicas). It isn't straight forward how to test this, as there seems to be a race between the background thread that flushes telemetry and actually querying for the counter's values on startup. Work towards #228. Release justification: Low risk monitoring improvement. Release note: None Co-authored-by: Rohan Yadav <rohany@cockroachlabs.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We need to be able to produce and apply snapshots so we can garbage collect the raft logs. This will work somewhat differently from MemoryStorage: We can synthesize snapshots on-demand in the Storage.Snapshot method instead of tying snapshot creation to an explicit Compact call. We can then advance Storage.FirstIndex whenever we discard old log entries.
The text was updated successfully, but these errors were encountered: