Github user vdiravka commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1059#discussion_r162292571
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestNestedLoopJoin.java
 ---
    @@ -19,19 +19,17 @@
     package org.apache.drill.exec.physical.impl.join;
     
     import org.apache.drill.categories.OperatorTest;
    -import org.apache.drill.PlanTestBase;
     import org.apache.drill.common.exceptions.UserRemoteException;
     import org.junit.BeforeClass;
     import org.junit.Test;
     import org.junit.experimental.categories.Category;
    -
     import java.nio.file.Paths;
    -
    +import org.apache.drill.exec.planner.physical.PlannerSettings;
     import static org.hamcrest.CoreMatchers.containsString;
     import static org.junit.Assert.assertThat;
     
     @Category(OperatorTest.class)
    -public class TestNestedLoopJoin extends PlanTestBase {
    +public class TestNestedLoopJoin extends JoinTestBase {
     
       private static String nlpattern = "NestedLoopJoin";
    --- End diff --
    
    Can you edit this as well? --> final, NLJ_PATTERN


---

Reply via email to