[GOBBLIN-97] shall Event Reporter and MetricReportReporter inherit from same class?

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Labels: question
• External issue URL: https://github.com/linkedin/gobblin/issues/1094
• Assignee: Unassigned
• Reporter: Abhishek Tiwari
• Created: 19/Jul/17 07:40
• Updated: 07/Aug/17 08:46

Description

Hi ,

I have a question regarding Event Reporter and Metric Reporter .
Since both the classes does the same task of reporting shouldn't they inherit from the same classes .
Right now the hierarchy is something like

KafkaReporter -> MetricReportReporter -> ConfiguredScheduledReporter -> ScheduledReporter(gobblin one) ->ContextAwareReporter

and

KafkaEventReporter -> EventReporter -> ScheduledReporter (codahale)

Github Url : https://github.com/linkedin/gobblin/issues/1094
Github Reporter : rahulmittal91
Github Assignee : ibuenros
Github Created At : 2016-07-11T12:35:28Z
Github Updated At : 2016-07-13T11:33:21Z