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

    https://github.com/apache/drill/pull/1059#discussion_r162292084
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/join/TestMergeJoinAdvanced.java
 ---
    @@ -38,13 +37,16 @@
     import java.util.Random;
     
     @Category(OperatorTest.class)
    -public class TestMergeJoinAdvanced extends BaseTestQuery {
    +public class TestMergeJoinAdvanced extends JoinTestBase {
       private static final String LEFT = "merge-join-left.json";
       private static final String RIGHT = "merge-join-right.json";
    +  private static String mjpattern = "MergeJoin";
    --- End diff --
    
    Make it as a constant --> private static final String MJ_PATTERN = 
"MergeJoin";


---

Reply via email to