Skip to content
Snippets Groups Projects
  • Marcelo Vanzin's avatar
    c53c902f
    [SPARK-9284] [TESTS] Allow all tests to run without an assembly. · c53c902f
    Marcelo Vanzin authored
    This change aims at speeding up the dev cycle a little bit, by making
    sure that all tests behave the same w.r.t. where the code to be tested
    is loaded from. Namely, that means that tests don't rely on the assembly
    anymore, rather loading all needed classes from the build directories.
    
    The main change is to make sure all build directories (classes and test-classes)
    are added to the classpath of child processes when running tests.
    
    YarnClusterSuite required some custom code since the executors are run
    differently (i.e. not through the launcher library, like standalone and
    Mesos do).
    
    I also found a couple of tests that could leak a SparkContext on failure,
    and added code to handle those.
    
    With this patch, it's possible to run the following command from a clean
    source directory and have all tests pass:
    
      mvn -Pyarn -Phadoop-2.4 -Phive-thriftserver install
    
    Author: Marcelo Vanzin <vanzin@cloudera.com>
    
    Closes #7629 from vanzin/SPARK-9284.
    c53c902f
    History
    [SPARK-9284] [TESTS] Allow all tests to run without an assembly.
    Marcelo Vanzin authored
    This change aims at speeding up the dev cycle a little bit, by making
    sure that all tests behave the same w.r.t. where the code to be tested
    is loaded from. Namely, that means that tests don't rely on the assembly
    anymore, rather loading all needed classes from the build directories.
    
    The main change is to make sure all build directories (classes and test-classes)
    are added to the classpath of child processes when running tests.
    
    YarnClusterSuite required some custom code since the executors are run
    differently (i.e. not through the launcher library, like standalone and
    Mesos do).
    
    I also found a couple of tests that could leak a SparkContext on failure,
    and added code to handle those.
    
    With this patch, it's possible to run the following command from a clean
    source directory and have all tests pass:
    
      mvn -Pyarn -Phadoop-2.4 -Phive-thriftserver install
    
    Author: Marcelo Vanzin <vanzin@cloudera.com>
    
    Closes #7629 from vanzin/SPARK-9284.