Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-05-30 Thread Vinzenz Feenstra
Hi, I finally have an update for the proposal ready, after some thinking and throwing some ideas around with colleagues we came to the conclusion that the best approach might be as followed (without having dynamic return values) I have set up some test engine with 'FakeVDSM' and I am

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-04-22 Thread Vinzenz Feenstra
...@redhat.com Cc: engine-de...@ovirt.org, vdsm-devel@lists.fedorahosted.org, Vinzenz Feenstra vfeen...@redhat.com Sent: Wednesday, March 13, 2013 5:39:24 PM Subject: Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics RetrievalOptimization - Original Message - I am completely

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-04-22 Thread Ayal Baron
] Proposal VDSM = Engine Data Statistics Retrieval Optimization - Original Message - I am completely against this. It make the return value differ according to input which is a big no no when talking about type safe APIs. The only reason we have this problem

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-17 Thread Ayal Baron
- Original Message - On 03/13/2013 11:55 PM, Ayal Baron wrote: ... The only reason we have this problem is because there is this thing against making multiple calls. Just split it up. getVmRuntimeStats() - transient things like mem and cpu% getVmInformation() - (semi)static

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-17 Thread Yaniv Kaul
On 17/03/13 15:13, Ayal Baron wrote: - Original Message - On 03/13/2013 11:55 PM, Ayal Baron wrote: ... The only reason we have this problem is because there is this thing against making multiple calls. Just split it up. getVmRuntimeStats() - transient things like mem and cpu%

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-17 Thread Yair Zaslavsky
- Original Message - From: Ayal Baron aba...@redhat.com To: Itamar Heim ih...@redhat.com Cc: engine-de...@ovirt.org, vdsm-devel@lists.fedorahosted.org Sent: Sunday, March 17, 2013 3:13:09 PM Subject: Re: [Engine-devel] [vdsm] Proposal VDSM = Engine Data Statistics Retrieval

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-17 Thread Ayal Baron
- Original Message - On 17/03/13 15:13, Ayal Baron wrote: - Original Message - On 03/13/2013 11:55 PM, Ayal Baron wrote: ... The only reason we have this problem is because there is this thing against making multiple calls. Just split it up. getVmRuntimeStats()

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-17 Thread Ayal Baron
- Original Message - - Original Message - From: Ayal Baron aba...@redhat.com To: Itamar Heim ih...@redhat.com Cc: engine-de...@ovirt.org, vdsm-devel@lists.fedorahosted.org Sent: Sunday, March 17, 2013 3:13:09 PM Subject: Re: [Engine-devel] [vdsm] Proposal VDSM =

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-13 Thread Saggi Mizrahi
I am completely against this. It make the return value differ according to input which is a big no no when talking about type safe APIs. The only reason we have this problem is because there is this thing against making multiple calls. Just split it up. getVmRuntimeStats() - transient things

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-13 Thread Ayal Baron
- Original Message - I am completely against this. It make the return value differ according to input which is a big no no when talking about type safe APIs. The only reason we have this problem is because there is this thing against making multiple calls. Just split it up.

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-13 Thread Saggi Mizrahi
VDSM = Engine Data Statistics Retrieval Optimization - Original Message - I am completely against this. It make the return value differ according to input which is a big no no when talking about type safe APIs. The only reason we have this problem is because

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-08 Thread Vinzenz Feenstra
On 03/08/2013 03:30 AM, Mark Wu wrote: On 03/08/2013 06:11 AM, Dan Kenigsberg wrote: On Thu, Mar 07, 2013 at 12:25:54PM +0100, Vinzenz Feenstra wrote: Please find the prettier version on the wiki: http://www.ovirt.org/Proposal_VDSM_-_Engine_Data_Statistics_Retrieval Proposal VDSM - Engine

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-08 Thread Dan Kenigsberg
On Fri, Mar 08, 2013 at 09:37:03AM +0100, Vinzenz Feenstra wrote: snip Improvement of the Guest Agent: As part of the proposed solution it is necessary to improve the guest agent as well. Improving the agent may be a good idea, but I do not see the necessity in it. The guest

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-08 Thread Dan Kenigsberg
On Fri, Mar 08, 2013 at 10:21:27AM +0800, Mark Wu wrote: On 03/07/2013 07:25 PM, Vinzenz Feenstra wrote: Please find the prettier version on the wiki: http://www.ovirt.org/Proposal_VDSM_-_Engine_Data_Statistics_Retrieval Proposal VDSM - Engine Data Statistics Retrieval VDSM =

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-07 Thread Dan Kenigsberg
On Thu, Mar 07, 2013 at 12:25:54PM +0100, Vinzenz Feenstra wrote: Please find the prettier version on the wiki: http://www.ovirt.org/Proposal_VDSM_-_Engine_Data_Statistics_Retrieval Proposal VDSM - Engine Data Statistics Retrieval VDSM = Engine data retrieval optimization

Re: [vdsm] [Engine-devel] Proposal VDSM = Engine Data Statistics Retrieval Optimization

2013-03-07 Thread Mark Wu
On 03/08/2013 06:11 AM, Dan Kenigsberg wrote: On Thu, Mar 07, 2013 at 12:25:54PM +0100, Vinzenz Feenstra wrote: Please find the prettier version on the wiki: http://www.ovirt.org/Proposal_VDSM_-_Engine_Data_Statistics_Retrieval Proposal VDSM - Engine Data Statistics Retrieval VDSM =