Ahh, good point. Not sure why it was done that way originally. Response.notFound should work.
Will fix.



> On Mar 7, 2014, at 9:22 PM, Tobi Knaup <notifications@github.com> wrote:
> 
> In src/main/scala/mesosphere/marathon/api/v1/TasksResource.scala:
> 
> > @@ -42,6 +40,7 @@ class TasksResource @Inject()(
> >        x.getHost == host || x.getId == id || host == "*"
> >      )
> >  
> > -    service.killTasks(appId, toKill, scale)
> > +    if (toKill.size == 0) Response.noContent.status(404).build
> Then we should fix it everywhere. It makes sense that it returns the correct code, since the 404 gets set last. But setting a 204 and overwriting it right away with a 404 is unnecessary.
> 
> —
> Reply to this email directly or view it on GitHub.