On Oct 31, 2013, at 4:48 AM, [email protected] wrote:

> +      /*
>       lmgmt->alarm_keeper->signalAlarm(MGMT_ALARM_PROXY_SYSTEM_ERROR,
>                                        "[TrafficManager] Unable to write 
> cluster.config, membership unchanged");
> +      */


If this is dead code, why leave it? Or if we’re uncertain, at least add a 
comment as to why the code is left commented out? Same with the rest of the 
patch.

— Leif

Reply via email to