Flume
  1. Flume
  2. FLUME-292

Refactor OutputFormat and OutputFormatBuilder to better support plugin authoring

    Details

      Description

      OutputFormat and OutputFormatBuilder usage has been inconsistent and required multiple places where information was repeated. Both should be refactored, as well as their consumers, such that:

      • Instantiation / building of OutputFormats is consistent within the code base.
      • OutputFormat names can be centralized and reused.
      • Each class conforms to the "do one thing and do it well" rule.

        Issue Links

          Activity

          E. Sammer created issue -
          E. Sammer made changes -
          Field Original Value New Value
          Status Open [ 1 ] Patch Available [ 10000 ]
          E. Sammer made changes -
          Link This issue depends on FLUME-195 [ FLUME-195 ]
          Hide
          E. Sammer added a comment -

          I can't post the review just yet because RB is complaining my flume-195 branch blobs aren't in the repo (which is true) yet. This patch is on top of the FLUME-195 patch set so I have to get that through first.

          Show
          E. Sammer added a comment - I can't post the review just yet because RB is complaining my flume-195 branch blobs aren't in the repo (which is true) yet. This patch is on top of the FLUME-195 patch set so I have to get that through first.
          Hide
          Disabled imported user added a comment -

          If you can push to a branch, I think RB will eventually pull it and you can upload the diff.

          Show
          Disabled imported user added a comment - If you can push to a branch, I think RB will eventually pull it and you can upload the diff.
          Hide
          E. Sammer added a comment -

          Thanks Henry. I did exactly that.

          https://review.cloudera.org/r/1085/

          Show
          E. Sammer added a comment - Thanks Henry. I did exactly that. https://review.cloudera.org/r/1085/
          Hide
          E. Sammer added a comment -

          Pushed to master.

          Show
          E. Sammer added a comment - Pushed to master.
          E. Sammer made changes -
          Status Patch Available [ 10000 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Mark Thomas made changes -
          Project Import Tue Aug 02 16:57:12 UTC 2011 [ 1312304232406 ]
          Arvind Prabhakar made changes -
          Fix Version/s v0.9.5 [ 12317558 ]

            People

            • Assignee:
              E. Sammer
              Reporter:
              E. Sammer
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development