Uploaded image for project: 'Apache Storm'
  1. Apache Storm
  2. STORM-2854

Expose IEventLogger to make event logging pluggable

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

Details

    Description

      For the first time, "Event Logger" feature is designed to make implementation pluggable, so that's why IEventLogger exists, but we didn't have actual use case other than just writing them to the file at that time, so we just simplified the case.

      Now we have use case which also write events to file, but with awareness of structure of event so that it can be easily parseable from log feeder. We would want to have custom IEventLogger to represent event as our own format in this case.

      There's another issue as well: EventInfo has `ts` which stores epoch but it's defined as String, not long.

      Attachments

        Activity

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

          People

            kabhwan Jungtaek Lim
            kabhwan Jungtaek Lim
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 3h 20m
                3h 20m

                Slack

                  Issue deployment