mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <vinodk...@gmail.com>
Subject Re: Review Request 41454: Added initial draft of executor HTTP API user doc.
Date Tue, 05 Jan 2016 00:20:31 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41454/#review112636
-----------------------------------------------------------



docs/executor-http-api.md (line 14)
<https://reviews.apache.org/r/41454/#comment173138>

    put 4 spaces so that you get a shaded block around this. see https://github.com/apache/mesos/blob/master/docs/scheduler-http-api.md
for reference.



docs/executor-http-api.md (line 30)
<https://reviews.apache.org/r/41454/#comment173139>

    s/Alternatively/Additonally/



docs/executor-http-api.md (line 31)
<https://reviews.apache.org/r/41454/#comment173140>

    "The executor is expected to keep running...". This sentence should be at the top of these
two sub points. Also, the executor should keep running only if checkpointing is enabled right?
    
    "These status update messages...". How about:
    "The executor is expected to keep track of status updates not acknowledged by the agent
via the ACKNOWLEDGE events."



docs/executor-http-api.md (line 32)
<https://reviews.apache.org/r/41454/#comment173141>

    there should be no network intermediaries between an agent and an executor!
    
    can you just re-use the comment in exec.cpp above `tasks` map instead?



docs/executor-http-api.md (lines 52 - 70)
<https://reviews.apache.org/r/41454/#comment173142>

    looking at just the JSON, it's not very clear that `tasks` and `updates` correspond to
unacknowledged tasks and updates. we should probably rename these fields as `unacknowledged_tasks`
and `unacknowledged_updates`. what do you think?



docs/executor-http-api.md (line 131)
<https://reviews.apache.org/r/41454/#comment173144>

    Is the retry by executor necessary? The current driver doesn't do retries for example.
Are you anticipating a scenario where the executor is connected but messages are lost?



docs/executor-http-api.md (line 171)
<https://reviews.apache.org/r/41454/#comment173145>

    s/message to the executor/message to the scheduler/



docs/executor-http-api.md (line 211)
<https://reviews.apache.org/r/41454/#comment173147>

    s/messages/tasks/



docs/executor-http-api.md (line 247)
<https://reviews.apache.org/r/41454/#comment173148>

    s/the resources/the task resources/



docs/executor-http-api.md (line 278)
<https://reviews.apache.org/r/41454/#comment173149>

    Mention that `data` is raw bytes encoded in Base64?



docs/executor-http-api.md (line 293)
<https://reviews.apache.org/r/41454/#comment173150>

    Also mention that TASK_LOST updates will be sent by the slave for any active tasks.



docs/executor-http-api.md (lines 332 - 333)
<https://reviews.apache.org/r/41454/#comment173192>

    didn't we agree to rename these?



docs/executor-http-api.md (line 335)
<https://reviews.apache.org/r/41454/#comment173193>

    s/all agent/all the agent's/



docs/executor-http-api.md (line 343)
<https://reviews.apache.org/r/41454/#comment173195>

    what's the relation between MESOS_RETRY_INTERVAL_MAX and the duration after which the
agent deems recovery process complete (EXECUTOR_REREGISTER_TIMEOUT)? AFAICT, currently the
former is set to 15 min and the latter is 2s.



docs/executor-http-api.md (line 350)
<https://reviews.apache.org/r/41454/#comment173194>

    s/then kill/then forcefully kill/
    
    s/in forcefully/in/


- Vinod Kone


On Dec. 17, 2015, 1:07 a.m., Anand Mazumdar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41454/
> -----------------------------------------------------------
> 
> (Updated Dec. 17, 2015, 1:07 a.m.)
> 
> 
> Review request for mesos and Vinod Kone.
> 
> 
> Bugs: MESOS-4177
>     https://issues.apache.org/jira/browse/MESOS-4177
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This is an initial draft for the user doc of Executor HTTP API. Some details might change
due to the pending executor library review chain: https://reviews.apache.org/r/41275
> 
> 
> Diffs
> -----
> 
>   docs/executor-http-api.md PRE-CREATION 
>   docs/home.md a01612ec8347eb8a7a9277b829365b6c3a1fe9e3 
> 
> Diff: https://reviews.apache.org/r/41454/diff/
> 
> 
> Testing
> -------
> 
> https://gist.github.com/hatred/d35fdaa667203bf34a78
> 
> 
> Thanks,
> 
> Anand Mazumdar
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message