Re: [Dev] [ESB] MediationTracerService is tightly coupled with MemoryAppender
We should not use appenders to read the logs back, there are set of log providers/readers available in platform for a eg : TenantAwareLogProvider/TenantAwareLogReader can be used to get logs from MemoryAppender. This log provider mechanism has been improved in 4.3.0 trunk to support pluggable log providers. Another thing is , if we disable MemoryAppender in production systems( due to memory issues) then the above components will not function. Relevant product teams, please change this and use Log providers to read logs which is the most clean way to get logs. Thanks, Shameera. On Tue, Jul 22, 2014 at 7:11 PM, Gayashan Amarasinghe wrote: > Hi, > > I observed that the same implementation is followed in > EventTracerAdminService [1] and JiraAppender [2] as well. > > [1] > https://github.com/wso2-dev/carbon-event-processing/blob/master/components/event-monitor/event-tracer/org.wso2.carbon.event.tracer/src/main/java/org/wso2/carbon/event/tracer/EventTracerAdminService.java > > [2] > https://github.com/wso2-dev/platform/blob/master/components/health-monitor/org.wso2.carbon.health/src/main/java/org/wso2/carbon/health/service/JiraAppender.java > > /Gayashan > > > On Thu, Jul 17, 2014 at 4:20 PM, Gayashan Amarasinghe > wrote: > >> Hi all, >> >> MediationTracerService [1] within the carbon-mediation repo is tightly >> coupled with the MemoryAppender which is available in the carbon-kernel. >> MediationTracerService is only reading the log. We believe this is not the >> best way to achieve this. We can use a Logging Provider [2] available >> within the carbon-utils to achieve this task. >> >> WDYT? >> >> >> [1] >> https://github.com/wso2-dev/carbon-mediation/blob/master/components/mediation-monitor/mediation-tracer/org.wso2.carbon.mediation.tracer/src/main/java/org/wso2/carbon/mediation/tracer/MediationTracerService.java >> >> >> [2] >> https://github.com/wso2-dev/carbon-utils/tree/master/components/logging/org.wso2.carbon.logging.service/src/main/java/org/wso2/carbon/logging/util >> >> Thanks. >> >> /Gayashan >> >> -- >> *Gayashan Amarasinghe* >> Software Engineer | Platform TG >> WSO2, Inc. | http://wso2.com >> lean. enterprise. middleware >> >> Mobile : +94718314517 >> Blog : gayashan-a.blogspot.com >> > > > > -- > *Gayashan Amarasinghe* > Software Engineer | Platform TG > WSO2, Inc. | http://wso2.com > lean. enterprise. middleware > > Mobile : +94718314517 > Blog : gayashan-a.blogspot.com > -- *Software Engineer - WSO2 Inc.* *email: shameera AT wso2.com , shameera AT apache.org * *phone: +9471 922 1454* *Linked in : *http://lk.linkedin.com/pub/shameera-rathnayaka/1a/661/561 *Twitter : *https://twitter.com/Shameera_R ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [ESB] MediationTracerService is tightly coupled with MemoryAppender
Hi, I observed that the same implementation is followed in EventTracerAdminService [1] and JiraAppender [2] as well. [1] https://github.com/wso2-dev/carbon-event-processing/blob/master/components/event-monitor/event-tracer/org.wso2.carbon.event.tracer/src/main/java/org/wso2/carbon/event/tracer/EventTracerAdminService.java [2] https://github.com/wso2-dev/platform/blob/master/components/health-monitor/org.wso2.carbon.health/src/main/java/org/wso2/carbon/health/service/JiraAppender.java /Gayashan On Thu, Jul 17, 2014 at 4:20 PM, Gayashan Amarasinghe wrote: > Hi all, > > MediationTracerService [1] within the carbon-mediation repo is tightly > coupled with the MemoryAppender which is available in the carbon-kernel. > MediationTracerService is only reading the log. We believe this is not the > best way to achieve this. We can use a Logging Provider [2] available > within the carbon-utils to achieve this task. > > WDYT? > > > [1] > https://github.com/wso2-dev/carbon-mediation/blob/master/components/mediation-monitor/mediation-tracer/org.wso2.carbon.mediation.tracer/src/main/java/org/wso2/carbon/mediation/tracer/MediationTracerService.java > > > [2] > https://github.com/wso2-dev/carbon-utils/tree/master/components/logging/org.wso2.carbon.logging.service/src/main/java/org/wso2/carbon/logging/util > > Thanks. > > /Gayashan > > -- > *Gayashan Amarasinghe* > Software Engineer | Platform TG > WSO2, Inc. | http://wso2.com > lean. enterprise. middleware > > Mobile : +94718314517 > Blog : gayashan-a.blogspot.com > -- *Gayashan Amarasinghe* Software Engineer | Platform TG WSO2, Inc. | http://wso2.com lean. enterprise. middleware Mobile : +94718314517 Blog : gayashan-a.blogspot.com ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
[Dev] [ESB] MediationTracerService is tightly coupled with MemoryAppender
Hi all, MediationTracerService [1] within the carbon-mediation repo is tightly coupled with the MemoryAppender which is available in the carbon-kernel. MediationTracerService is only reading the log. We believe this is not the best way to achieve this. We can use a Logging Provider [2] available within the carbon-utils to achieve this task. WDYT? [1] https://github.com/wso2-dev/carbon-mediation/blob/master/components/mediation-monitor/mediation-tracer/org.wso2.carbon.mediation.tracer/src/main/java/org/wso2/carbon/mediation/tracer/MediationTracerService.java [2] https://github.com/wso2-dev/carbon-utils/tree/master/components/logging/org.wso2.carbon.logging.service/src/main/java/org/wso2/carbon/logging/util Thanks. /Gayashan -- *Gayashan Amarasinghe* Software Engineer | Platform TG WSO2, Inc. | http://wso2.com lean. enterprise. middleware Mobile : +94718314517 Blog : gayashan-a.blogspot.com ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev