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 <dmclaugh...@apache.org>
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 <mohit.ja...@uber.com>
> 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?
>>
>>
>>
>
