8000 [Java] Add support for changing term buffer length and MTU in the Cluster log. by vyazelenko · Pull Request #1233 · aeron-io/aeron · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Java] Add support for changing term buffer length and MTU in the Cluster log. #1233

< 8000 div class="d-flex flex-order-2 flex-md-order-1 mx-2">
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

Merged
merged 7 commits into from
Sep 27, 2021

Conversation

vyazelenko
Copy link
Contributor

This PR adds a feature that allows a Cluster to be restarted with the term buffer length and/or MTU changed. This requires truncating an existing recording log to the latest valid snapshot, i.e. Cluster is starting from a snapshot only. The new API to do such truncation is provided via the ClusterTool (i.e. io.aeron.cluster.ClusterTool#truncateRecordingLog and truncate-recording-log command).

Besides this feature this PR also contains a fix for keeping entries properly sorted in the RecodingLog.

@mjpt777
Copy link
Contributor
mjpt777 commented Sep 25, 2021

Using the term "truncate" on a recording log is confusing. It implies the log on the archive is truncated. We need to make it clear that this function creates a new recording log, old one is backed up, that allows the cluster to restart from the latest snapshot with a new cluster log. The old log remains untouched and can be archived or deleted as a separate operation. Something like "rebase log from snapshot.

…` and backup existing file before creating a new recording log. Clarify that the old file is available under new name `recording.log.bak`.
@vyazelenko
Copy link
Contributor Author

@mjpt777 I pushed a change that renamed truncateRecordingLog to seedRecordingLogFromSnapshot (seed-recording-log-from-snapshot) and adds a backup of original recording log as recording.log.bak file. JavaDoc updated to reflect these changes.

@mjpt777 mjpt777 merged commit 2f1b939 into aeron-io:master Sep 27, 2021
@vyazelenko vyazelenko deleted the cluster-tem-length-change branch September 27, 2021 09:16
mjpt777 added a commit that referenced this pull request Sep 27, 2021
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.

2 participants
0