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

    https://github.com/apache/drill/pull/1045#discussion_r162495937
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/test/OperatorFixture.java ---
    @@ -127,20 +156,26 @@ public OperatorFixture build() {
        * uses the same code generation mechanism as the full Drill, but
        * provide test-specific versions of various other services.
        */
    -
    -  public static class TestFragmentContext extends BaseFragmentContext {
    -
    +  public static class MockFragmentContext extends BaseFragmentContext {
    --- End diff --
    
    I like prefixing Mock classes with **Mock** instead of **Test**.


---

Reply via email to