Re: [MERGE] Merge request: api_refactoring on master

2013-01-10 Thread Prasanna Santhanam
On Wed, Jan 09, 2013 at 05:45:03PM -0500, Min Chen wrote: Hi Marcus, To answer your question below: Does this mean we won't see 10 SELECT uuid FROM `vm_instance` WHERE id=3079 OR uuid='3079' type calls for every VM we have when doing listVirtualMachines and such? Yes, that is exactly one

Re: [MERGE] Merge request: api_refactoring on master

2013-01-10 Thread Prasanna Santhanam
On Thu, Jan 10, 2013 at 03:57:49PM +0530, Prasanna Santhanam wrote: On Wed, Jan 09, 2013 at 05:45:03PM -0500, Min Chen wrote: Hi Marcus, To answer your question below: Does this mean we won't see 10 SELECT uuid FROM `vm_instance` WHERE id=3079 OR uuid='3079' type calls for every VM we

Re: [MERGE] Merge request: api_refactoring on master

2013-01-10 Thread Min Chen
Yes, I did a commit yesterday to completely get rid of IdentityProxy and IdentityTypeAdapter class to keep new developers from wrongly using them again in newly introduced Response class. Thanks -min On 1/10/13 2:31 AM, Prasanna Santhanam t...@apache.org wrote: On Thu, Jan 10, 2013 at

Re: [MERGE] Merge request: api_refactoring on master

2013-01-09 Thread Min Chen
...@citrix.commailto:rohit.ya...@citrix.com Subject: RE: [MERGE] Merge request: api_refactoring on master Date: 9 January 2013 11:36:47 AM PST To: Marcus Sorensen shadow...@gmail.commailto:shadow...@gmail.com No, so this change will first of all check the kind of input param we got as part

Re: [MERGE] Merge request: api_refactoring on master

2013-01-08 Thread Rohit Yadav
? Regards. From: Chip Childers [chip.child...@sungard.com] Sent: Monday, January 07, 2013 7:58 PM To: cloudstack-dev@incubator.apache.org Subject: Re: [MERGE] Merge request: api_refactoring on master On Sun, Jan 6, 2013 at 10:02 PM, Rohit Yadav rohit.ya

RE: [MERGE] Merge request: api_refactoring on master

2013-01-08 Thread Rohit Yadav
api_refactoring was merged successfully by a fast forward merge as the branch was already in sync with master: commit 077f47193d773e4fd193366117a75d45869d62c5 gpg: Signature made Tue Jan 8 20:36:41 2013 PST using RSA key ID 0EE3D884 Merge: d613c59 4f98d87 Author: Rohit Yadav bhais...@apache.org

RE: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Rohit Yadav
07, 2013 1:02 PM To: cloudstack-dev@incubator.apache.org Subject: RE: [MERGE] Merge request: api_refactoring on master Hey Rohit, Nice job! Is it possible to make a list of all the api calls that are be affected by this change and which have been tested? For example I like to know if I need

Re: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Chip Childers
On Sun, Jan 6, 2013 at 10:02 PM, Rohit Yadav rohit.ya...@citrix.com wrote: This is merge request for merging api_refactoring branch on master (https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring). For complete list of changes are provided at the end of the email.

Re: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread David Nalley
On Sun, Jan 6, 2013 at 10:02 PM, Rohit Yadav rohit.ya...@citrix.com wrote: This is merge request for merging api_refactoring branch on master (https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring). For complete list of changes are provided at the end of the email.

RE: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Rohit Yadav
discoverer for cloudmonkey) Regards. From: Chip Childers [chip.child...@sungard.com] Sent: Monday, January 07, 2013 7:58 PM To: cloudstack-dev@incubator.apache.org Subject: Re: [MERGE] Merge request: api_refactoring on master On Sun, Jan 6, 2013 at 10:02 PM

Re: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Chip Childers
the Javelin merge. Do you folks have a proposed schedule for making it happen? Regards. From: Chip Childers [chip.child...@sungard.com] Sent: Monday, January 07, 2013 7:58 PM To: cloudstack-dev@incubator.apache.org Subject: Re: [MERGE] Merge request

RE: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Rohit Yadav
window in mind) Regards. From: Chip Childers [chip.child...@sungard.com] Sent: Monday, January 07, 2013 10:17 PM To: cloudstack-dev@incubator.apache.org Subject: Re: [MERGE] Merge request: api_refactoring on master On Mon, Jan 7, 2013 at 11:38 AM, Rohit Yadav

Re: [MERGE] Merge request: api_refactoring on master

2013-01-07 Thread Rohit Yadav
On 07-Jan-2013, at 7:59 AM, David Nalley da...@gnsa.us wrote: On Sun, Jan 6, 2013 at 10:02 PM, Rohit Yadav rohit.ya...@citrix.com wrote: This is merge request for merging api_refactoring branch on master (https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+API+refactoring).

Re: [MERGE] Merge request: api_refactoring on master

2013-01-06 Thread Min Chen
Thanks Rohit. Added some details on list API performance optimization item below inline. -min On 1/6/13 7:02 PM, Rohit Yadav rohit.ya...@citrix.commailto:rohit.ya...@citrix.com wrote: This is merge request for merging api_refactoring branch on master

RE: [MERGE] Merge request: api_refactoring on master

2013-01-06 Thread Hugo Trippaers
[mailto:min.c...@citrix.com] Sent: Monday, January 07, 2013 6:40 AM To: cloudstack-dev@incubator.apache.org Subject: Re: [MERGE] Merge request: api_refactoring on master Thanks Rohit. Added some details on list API performance optimization item below inline. -min On 1/6/13 7:02 PM, Rohit