> --- TODO: add verifications to this call; the client should have a lock
> --- on the name of the instance, and no instance with the given uuid should
> --- exist.

Here you're removing a TODO with taking care of it; it is not verified
that the client owns a lock on the instance name. While it is perfectly
OK to implement only parts of a pending TODO, in this case only the
implemented parts should be removed from the TODO entry.

-- 
Klaus Aehlig
Google Germany GmbH, Dienerstr. 12, 80331 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Graham Law, Christine Elizabeth Flores

Reply via email to