Re: Review Request: Adding more Granular Global Parameters

2013-04-29 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10740/ --- (Updated April 29, 2013, 6:40 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request: Adding more Granular Global Parameters

2013-04-28 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10740/#review19830 --- server/src/com/cloud/configuration/Config.java

Re: Review Request: Adding more Granular Global Parameters

2013-04-25 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10740/#review19701 --- Can you resubmit the patch after changing scope_id/scope_name to

Re: Review Request: Adding more Granular Global Parameters

2013-04-25 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10740/ --- (Updated April 26, 2013, 3:37 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request: Adding more Granular Global Parameters

2013-04-25 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10740/ --- (Updated April 26, 2013, 5:31 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request: Adding more Granular Global Parameters

2013-04-25 Thread Prasanna Santhanam
On Fri, Apr 26, 2013 at 04:54:45AM -, Sateesh Chodapuneedi wrote: api/src/com/cloud/configuration/ConfigurationService.java https://reviews.apache.org/r/10740/#comment40757 Need to avoid * imports per coding guidelines. Likely coming from the IDE auto importing. I had to fix this in