-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Add logging for task stop operations #14192
Add logging for task stop operations #14192
Conversation
indexing-service/src/main/java/org/apache/druid/indexing/worker/executor/ExecutorLifecycle.java
Outdated
Show resolved
Hide resolved
.onHttpError(e -> Either.value(false)) | ||
.onNotAvailable(e -> Either.value(false)) | ||
.onHttpError(e -> { | ||
log.warn("Task [%s] coundln't be stopped because of http request failure.", id); |
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.
we also want to log what this failure was. is it something that e.getMessage will have?
return Either.value(false); | ||
}) | ||
.onNotAvailable(e -> { | ||
log.warn("Task [%s] coundln't be stopped because it is not available.", id); |
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.
what does being "not available" exactly mean here? The task port is not reachable?
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 corresponds to ServiceNotAvailableException
which is thrown when the service locator returns empty set of locations
return isSuccess(response); | ||
boolean isSuccess = isSuccess(response); | ||
if (!isSuccess) { | ||
log.warn("Task [%s] coundln't be stopped because of http request failure.", id); |
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.
is it possible to get the exact http request failure?
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.
Added
No description provided.