looks nice, it will be backed by the current query for one id? or will you write a findByIds()?
On Thu, Feb 6, 2014 at 9:35 AM, Abhinandan Prateek <abhinandan.prat...@citrix.com> wrote: > +1, The listVM call is one of the most resource intensive call. Any step > to optimise it are welcome. > > On 06/02/14 2:01 pm, "Koushik Das" <koushik....@citrix.com> wrote: > >>Currently list VM can only be called using a single VM ID. So if there is >>a need to query a set of VMs using ID then either multiple list VM calls >>need to be made or all VMs needs to be fetched and then do a client side >>filtering. Both approaches are sub-optimal - the former results in >>multiple queries to database and the latter will be an overkill if you >>need a small subset from a very large number of VMs. >> >>The proposal is to have an additional parameter to specify a list of VM >>IDs for which the data needs to be fetched. Using this the required VMs >>can be queried in an efficient manner. With the new parameter the syntax >>would look like >>http://localhost:8096/api?command=listVirtualMachines&listAll=true&ids=edd >>ac053-9b12-4d2e-acb7-233de2e98112,009966fc-4d7b-4f84-8609-254979ba0134 >> >>The new 'ids' parameter will be mutually exclusive with the existing 'id' >>parameter. >> >>Let me know if there are any concerns/comments. >> >>Thanks, >>Koushik > -- Daan
