Hi Vishanth, We can remove the check of log.isInfoAvailable() too if (log.isInfoEnabled()) {log.info(String.format("Adding autoscaling policy: [id] %s", policy.getId()));}
On Fri, Mar 6, 2015 at 6:13 PM, Vishanth Balasubramaniam <vishan...@wso2.com > wrote: > Hi Devs, > > I was working on throwing an exception in registry manager when persisting > autoscaler with an invalid autoscale policy id. > > While testing the scenario, I found out that the autoscaler policy is > added to the information model in policy manager, even if persisting > autoscaler policy is unsuccessful. > > Changes were made in the Pull Request #261 [1]. Please review and merge. > > [1] - https://github.com/apache/stratos/pull/261 > > Thank you, > Vishanth > > -- > *Vishanth Balasubramaniam* > Software Engineer > WSO2 Inc.; http://wso2.com > lean.enterprise.middleware > > mobile: *+94771737718* > about me: *http://about.me/vishanth <http://about.me/vishanth>* > > -- Udara Liyanage Software Engineer WSO2, Inc.: http://wso2.com lean. enterprise. middleware web: http://udaraliyanage.wordpress.com phone: +94 71 443 6897