your a champion On Thu, Feb 6, 2014 at 12:24 PM, Koushik Das <koushik....@citrix.com> wrote: > Yes it will be like a findByIds() and the one id case is just a special case > for this. > > On 06-Feb-2014, at 4:24 PM, Daan Hoogland <daan.hoogl...@gmail.com> > wrote: > >> 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 >
-- Daan
