Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-576

Properly reset FlumeBuilder SourceFactory after testing

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: v0.9.3
    • Fix Version/s: v0.9.5
    • Component/s: Test
    • Labels:
      None

      Description

      In com.cloudera.flume.agent.TestFlumeNode#testRestartNextException():

      FlumeBuilder.setSourceFactory(new SourceFactory() {
        public EventSource getSource(Context ctx, String name, String... args) {
          return new NextExnSource();
        }
      }
      

      This pollutes the JVM for all future tests leading to spurious errors in that it returns NextExnSource for any source that gets constructed.

        Issue Links

          Activity

          Hide
          jmhsieh Jonathan Hsieh added a comment -

          Better yet, FlumeBuilder should be made an object that is instantiated for each test. (removing any static state..)

          Show
          jmhsieh Jonathan Hsieh added a comment - Better yet, FlumeBuilder should be made an object that is instantiated for each test. (removing any static state..)
          Hide
          esammer E. Sammer added a comment -

          Removing myself from this. Not going to have time to focus on it.

          Show
          esammer E. Sammer added a comment - Removing myself from this. Not going to have time to focus on it.
          Hide
          paliwalashish Ashish Paliwal added a comment -

          Won't fix. 0.X branch not maintained anymore

          Show
          paliwalashish Ashish Paliwal added a comment - Won't fix. 0.X branch not maintained anymore

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development