kfaraz commented on code in PR #20417:
URL: https://github.com/apache/druid/pull/20417#discussion_r4105296275
##########
indexing-service/src/main/java/org/apache/druid/indexing/overlord/hrtr/HttpRemoteTaskRunner.java:
##########
@@ -1437,16 +1438,17 @@ public void taskAddedOrUpdated(final TaskAnnouncement
announcement, final Worker
taskItem = tasks.get(taskId);
if (taskItem == null) {
// Try to find information about it in the TaskStorage
- Optional<TaskStatus> knownStatusInStorage =
taskStorage.getStatus(taskId);
+ final TaskInfo taskInfo = taskStorage.getTaskInfo(taskId);
Review Comment:
We can ignore this for now. If payload is not deserializable, there would be
issues running the task as well. So it is likely to not be in a RUNNING state
(unless there is some specific bug where only the Overlord is unable to
deserialize it due to backward compat issues etc.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
