Re: [Dev] Unnecessary error message printing when realtime dashboard not displaying

2015-08-02 Thread Tharik Kanaka
Hi Damith, I did above change, please review and merge the pull request [1] [1] https://github.com/wso2/carbon-analytics-common/pull/90 Regards, On Mon, Aug 3, 2015 at 8:40 AM, Damith Wickramasinghe dami...@wso2.com wrote: Hi, +1. for printing it for only once. I will make the change.

[Dev] Unnecessary error message printing when realtime dashboard not displaying

2015-08-02 Thread Gihan Anuruddha
Hi CEP team, When we have a real time dashboard and dashboard not display in the browser, below message is continuously printing. I think we can't assume always realtime dashboards are displaying in the browser. So these redundant messages are unnecessarly filling the logs. I think one error

Re: [Dev] Unnecessary error message printing when realtime dashboard not displaying

2015-08-02 Thread Sriskandarajah Suhothayan
looping Tharik and Damith On Sun, Aug 2, 2015 at 9:06 PM, Gihan Anuruddha gi...@wso2.com wrote: Hi CEP team, When we have a real time dashboard and dashboard not display in the browser, below message is continuously printing. I think we can't assume always realtime dashboards are displaying

Re: [Dev] Unnecessary error message printing when realtime dashboard not displaying

2015-08-02 Thread Tharik Kanaka
Hi Gihan, Yes it would be better to restrict printing error only once. Otherwise it would fill the console with redundant logs as you have mentioned. Regards, On Sun, Aug 2, 2015 at 11:12 PM, Sriskandarajah Suhothayan s...@wso2.com wrote: looping Tharik and Damith On Sun, Aug 2, 2015 at

Re: [Dev] Unnecessary error message printing when realtime dashboard not displaying

2015-08-02 Thread Damith Wickramasinghe
Hi, +1. for printing it for only once. I will make the change. Thanks, Damith. On Mon, Aug 3, 2015 at 7:23 AM, Tharik Kanaka tha...@wso2.com wrote: Hi Gihan, Yes it would be better to restrict printing error only once. Otherwise it would fill the console with redundant logs as you have