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
Recovery: add total operations to the _recovery
API
#10042
Conversation
This commit adds the current total number of translog operations to the recovery reporting API. We also expose the recovered / total percentage: ``` "translog": { "recovered": 536, "total": 986, "percent": "54.3%", "total_time": "2ms", "total_time_in_millis": 2 }, ``` Closes elastic#9368
@@ -331,6 +332,7 @@ public void testStageSequenceEnforcement() { | |||
assertThat(state.getStage(), equalTo(Stage.DONE)); | |||
} | |||
|
|||
@Repeat(iterations = 300) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this just for local beasting or you mean to push it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah. slipped in with the latest commit. will remove. thx,
LGTM |
@@ -493,51 +493,105 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws | |||
} | |||
|
|||
public static class Translog extends Timer implements ToXContent, Streamable { | |||
private final AtomicInteger currentTranslogOperations = new AtomicInteger(); | |||
public static int UNKNOWN = -1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this must be final!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OOPS! fixing..
@bleskes something minor that I saw, I think the percent should be a number, so it can be plotted for example, so: |
@s1monw pushed another round |
LGTM |
This commit adds the current total number of translog operations to the recovery reporting API. We also expose the recovered / total percentage: ``` "translog": { "recovered": 536, "total": 986, "percent": "54.3%", "total_time": "2ms", "total_time_in_millis": 2 }, ``` Closes elastic#9368 Closes elastic#10042
This commit adds the current total number of translog operations to the recovery reporting API. We also expose the recovered / total percentage:
Closes #9368