Re: shutdown vs kill API is Mesos

2018-01-12 Thread Mohit Jaggi
I understand. You don't agree with the second point of the summary. What
about this:

If I change Driver.kill it to have a method Driver.destroy that calls
either KILL or SHUTDOWN as follows:

void destroy(taskId, executorId, agentId) {

if(driver instanceOf Versioned)
   driver.shutdown(executorId, agentId)
else
   driver.kill(taskId)

}

Note the change in the signature to include 2 more params...

Any other ideas?


On Fri, Jan 12, 2018 at 11:39 AM, David McLaughlin 
wrote:

> I'm not sure I agree with the summary. Bill's proposal was using shutdown
> only when using the new API. I would also support this if it's possible.
>
> On Fri, Jan 12, 2018 at 11:14 AM, Mohit Jaggi 
> wrote:
>
>> Summary so far:
>> - Bill supports making this change
>> - This change cannot be made in a backward compatible manner
>> - David (Twitter) does not want to use HTTP APIs due to performance
>> concerns. I conclude that folks from Twitter don't support this change
>>
>> Question:
>> - Are there other users that want this change?
>>
>>
>>
>


Re: shutdown vs kill API is Mesos

2018-01-12 Thread David McLaughlin
I'm not sure I agree with the summary. Bill's proposal was using shutdown
only when using the new API. I would also support this if it's possible.

On Fri, Jan 12, 2018 at 11:14 AM, Mohit Jaggi  wrote:

> Summary so far:
> - Bill supports making this change
> - This change cannot be made in a backward compatible manner
> - David (Twitter) does not want to use HTTP APIs due to performance
> concerns. I conclude that folks from Twitter don't support this change
>
> Question:
> - Are there other users that want this change?
>
>
>


Re: shutdown vs kill API is Mesos

2018-01-12 Thread Mohit Jaggi
Summary so far:
- Bill supports making this change
- This change cannot be made in a backward compatible manner
- David (Twitter) does not want to use HTTP APIs due to performance
concerns. I conclude that folks from Twitter don't support this change

Question:
- Are there other users that want this change?