done changes as per the comments made from the code review

On Wed, Oct 22, 2014 at 6:39 PM, Harsha Kumara <notifications@github.com>
wrote:

> In
> components/cassandra/org.wso2.carbon.cassandra.cluster.mgt/src/main/java/org/wso2/carbon/cassandra/cluster/mgt/component/ClusterAdminComponentManager.java:
>
> >          try {
> >              startMonitoring();
> >          } catch (ClusterDataAdminException e) {
> > -           if(log.isDebugEnabled())
> > -           {
> > -               log.error("Error while starting cluster monitoring",e);
> > -           }
> > +            log.error("Error while starting cluster monitoring", e);
>
> Since this will happen at the component initialization, it will cause
> whole component initialization fail. So make it as error message and not re
> throw again.
>
+1 for this approach.


> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wso2-dev/carbon-storage-management/pull/49/files#r19211721>
> .
>



-- 
Prabath