Uploaded image for project: 'Apache Gobblin'
  1. Apache Gobblin
  2. GOBBLIN-97

shall Event Reporter and MetricReportReporter inherit from same class?

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None

    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 : Issac Buenrostro
      Github Created At : 2016-07-11T12:35:28Z
      Github Updated At : 2016-07-13T11:33:21Z

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            abti Abhishek Tiwari

            Dates

              Created:
              Updated:

              Slack

                Issue deployment