[MESOS-5276] HTTPCommandExecutor should terminate after it receives an ACK from the agent

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Epic Link: Executor API v1 improvements
• Assignee: Qian Zhang
• Reporter: Vinod Kone
• Created: 25/Apr/16 20:34
• Updated: 23/Sep/16 01:15
• Resolved: 23/Sep/16 01:15

Description

Currently, the HTTP command executor does a os::sleep() (similar to the driver based command executor) after sending the terminal update as a hack to ensure the update is received by the agent. The right thing to do would be to terminate after receiving the ACK from the agent.

For this to work we need to ensure the agent always ACKs an executor update (MESOS-5262).

Issue Links

• Blocked: MESOS-5262 slave::statusUpdate() should always ACK the status update sent by executor
• is related to: MESOS-2346 Docker tasks exiting normally, but returning TASK_FAILED
• is related to: MESOS-243 driver stop() should block until outstanding requests have been persisted
• is related to: MESOS-6026 Tasks mistakenly marked as FAILED due to race b/w sendExecutorTerminatedStatusUpdate() and _statusUpdate().