Flume
  1. Flume
  2. FLUME-576

Properly reset FlumeBuilder SourceFactory after testing

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major 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
          Ashish Paliwal added a comment -

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

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

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

          Show
          E. Sammer added a comment - Removing myself from this. Not going to have time to focus on it.
          Hide
          Jonathan Hsieh added a comment -

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

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development